| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 22 Aug 2023 18:56:53 +0530
From: "Somalapuram, Amaranath" <asomalap@....com>
To: Shashank Sharma <shashank.sharma@....com>,
Christian König <christian.koenig@....com>,
Christophe JAILLET <christophe.jaillet@...adoo.fr>,
Arunpravin.PaneerSelvam@....com,
"Prosyak, Vitaly" <Vitaly.Prosyak@....com>,
"Somalapuram, Amaranath" <Amaranath.Somalapuram@....com>
Cc: kernel-janitors@...r.kernel.org, dri-devel@...ts.freedesktop.org,
amd-gfx@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
Felix.Kuehling@....com, daniel@...ll.ch, airlied@...il.com,
Xinhui.Pan@....com, alexander.deucher@....com
Subject: Re: [PATCH 0/4] drm/amdgpu: Explicitly add a flexible array at the
end of 'struct amdgpu_bo_list' and simplify amdgpu_bo_list_create()
On 8/21/2023 6:30 PM, Shashank Sharma wrote:
> + Amar should be able to help.
>
> Amar,
>
> Can you please check this patch (series if required) with a few IGTs
> and probably with Xonotic as well ?
>
Tested patch series with IGT and Xonoti,
1st time I observed while launching xonoti, some kernel WARNING. But
unable to reproduced it.
Attaching dmesg.
[ 739.564460] ------------[ cut here ]------------
[ 739.564466] amdgpu 0000:03:00.0:
drm_WARN_ON_ONCE(drm_drv_uses_atomic_modeset(dev))
[ 739.564518] WARNING: CPU: 15 PID: 1072 at
drivers/gpu/drm/drm_vblank.c:728
drm_crtc_vblank_helper_get_vblank_timestamp_internal+0x375/0x3d0 [drm]
[ 739.564593] Modules linked in: amdgpu snd_hda_codec_realtek
snd_hda_codec_generic ledtrig_audio snd_hda_codec_hdmi intel_rapl_msr
intel_rapl_common snd_hda_intel edac_mce_amd binfmt_misc
snd_intel_dspcfg snd_intel_sdw_acpi snd_hda_codec iommu_v2 snd_hda_core
drm_buddy kvm_amd snd_hwdep gpu_sched nls_iso8859_1 snd_pcm
drm_suballoc_helper drm_ttm_helper kvm ttm snd_seq_midi
drm_display_helper snd_seq_midi_event snd_rawmidi cec rc_core snd_seq
crct10dif_pclmul ghash_clmulni_intel drm_kms_helper sha512_ssse3
aesni_intel snd_seq_device i2c_algo_bit crypto_simd snd_timer
snd_rn_pci_acp3x syscopyarea cryptd sysfillrect snd_acp_config
snd_soc_acpi joydev input_leds rapl wmi_bmof sysimgblt snd snd_pci_acp3x
ccp k10temp soundcore mac_hid amd_pmc sch_fq_codel msr parport_pc ppdev
lp drm parport ramoops reed_solomon pstore_blk pstore_zone efi_pstore
ip_tables x_tables autofs4 uas usb_storage hid_generic nvme usbhid hid
crc32_pclmul i2c_piix4 r8169 nvme_core ahci xhci_pci libahci realtek
xhci_pci_renesas video wmi gpio_amdpt
[ 739.564703] gpio_generic
[ 739.564707] CPU: 15 PID: 1072 Comm: gnome-shell Not tainted 6.3.0-rc4+ #5
[ 739.564713] Hardware name: AMD Artic/Artic-RN, BIOS WAF2309N 03/07/2022
[ 739.564716] RIP:
0010:drm_crtc_vblank_helper_get_vblank_timestamp_internal+0x375/0x3d0 [drm]
[ 739.564779] Code: 4c 8b 67 50 4d 85 e4 75 03 4c 8b 27 e8 44 23 d9 d3
48 c7 c1 58 7a 62 c0 4c 89 e2 48 c7 c7 74 a4 62 c0 48 89 c6 e8 cb 12 4c
d3 <0f> 0b eb 85 4c 8b 65 98 44 8b 5d 94 41 89 c5 4d 85 e4 74 05 4d 8b
[ 739.564782] RSP: 0018:ffffbde6c172faf0 EFLAGS: 00010282
[ 739.564787] RAX: 0000000000000000 RBX: ffffffffc0e89370 RCX:
0000000000000000
[ 739.564790] RDX: 0000000000000002 RSI: ffffffff950ca531 RDI:
00000000ffffffff
[ 739.564793] RBP: ffffbde6c172fb60 R08: 0000000000000000 R09:
00000000ffffdfff
[ 739.564795] R10: ffffbde6c172f958 R11: ffffffff95556ec8 R12:
ffff9d4881cd0820
[ 739.564798] R13: 0000000000000000 R14: 0000000000000000 R15:
ffff9d4886b9b8a8
[ 739.564800] FS: 00007fac3d72c5c0(0000) GS:ffff9d4da5fc0000(0000)
knlGS:0000000000000000
[ 739.564804] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 739.564807] CR2: 000055e4351b4a38 CR3: 0000000122ad8000 CR4:
0000000000350ee0
[ 739.564810] Call Trace:
[ 739.564813] <TASK>
[ 739.564817] ? dma_resv_get_singleton+0xb7/0x130
[ 739.564828] drm_crtc_vblank_helper_get_vblank_timestamp+0x20/0x30 [drm]
[ 739.564889] drm_crtc_get_last_vbltimestamp+0x59/0x90 [drm]
[ 739.564950] drm_crtc_next_vblank_start+0x44/0x80 [drm]
[ 739.565010] drm_atomic_helper_wait_for_fences+0x87/0x1e0
[drm_kms_helper]
[ 739.565045] drm_atomic_helper_commit+0xa1/0x160 [drm_kms_helper]
[ 739.565076] drm_atomic_commit+0x9d/0xd0 [drm]
[ 739.565172] ? __pfx___drm_printfn_info+0x10/0x10 [drm]
[ 739.565240] drm_mode_atomic_ioctl+0xa0b/0xba0 [drm]
[ 739.565306] ? __pfx_drm_mode_atomic_ioctl+0x10/0x10 [drm]
[ 739.565381] drm_ioctl_kernel+0xbf/0x150 [drm]
[ 739.565474] drm_ioctl+0x29e/0x500 [drm]
[ 739.565539] ? __pfx_drm_mode_atomic_ioctl+0x10/0x10 [drm]
[ 739.565606] ? _raw_spin_unlock_irqrestore+0x2b/0x50
[ 739.565614] amdgpu_drm_ioctl+0x52/0x90 [amdgpu]
[ 739.566064] __x64_sys_ioctl+0x99/0xd0
[ 739.566073] do_syscall_64+0x3f/0x90
[ 739.566079] entry_SYSCALL_64_after_hwframe+0x72/0xdc
[ 739.566086] RIP: 0033:0x7fac40b1aaff
[ 739.566091] Code: 00 48 89 44 24 18 31 c0 48 8d 44 24 60 c7 04 24 10
00 00 00 48 89 44 24 08 48 8d 44 24 20 48 89 44 24 10 b8 10 00 00 00 0f
05 <41> 89 c0 3d 00 f0 ff ff 77 1f 48 8b 44 24 18 64 48 2b 04 25 28 00
[ 739.566094] RSP: 002b:00007ffc71f5c250 EFLAGS: 00000246 ORIG_RAX:
0000000000000010
[ 739.566099] RAX: ffffffffffffffda RBX: 00007ffc71f5c2f0 RCX:
00007fac40b1aaff
[ 739.566102] RDX: 00007ffc71f5c2f0 RSI: 00000000c03864bc RDI:
0000000000000009
[ 739.566104] RBP: 00000000c03864bc R08: 0000000000000010 R09:
0000000000000010
[ 739.566107] R10: 0000000000000007 R11: 0000000000000246 R12:
000055e43609c5d0
[ 739.566110] R13: 0000000000000009 R14: 000055e43488cac0 R15:
000055e435f14ee0
[ 739.566116] </TASK>
[ 739.566118] ---[ end trace 0000000000000000 ]---
>
> Regards
>
> Shashank
>
> On 21/08/2023 13:03, Christian König wrote:
>> Am 20.08.23 um 11:51 schrieb Christophe JAILLET:
>>> This serie simplifies amdgpu_bo_list_create() and usage of the 'struct
>>> amdgpu_bo_list'.
>>
>> Oh, yes please. That's something I always wanted to cleanup as well.
>>
>>> It is compile tested only.
>>
>> That bothers me a bit. Arun, Vitaly, Shashank can anybody with time
>> run a quick test on this? Preferable the IGT tests, but this just
>> some game should probably do as well.
>>
>> Apart from that looks good to me,
>> Christian.
>>
>>>
>>> Christophe JAILLET (4):
>>> drm/amdgpu: Explicitly add a flexible array at the end of 'struct
>>> amdgpu_bo_list'
>>> drm/amdgpu: Remove a redundant sanity check
>>> drm/amdgpu: Remove amdgpu_bo_list_array_entry()
>>> drm/amdgpu: Use kvzalloc() to simplify code
>>>
>>> drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.c | 15 ++-------------
>>> drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.h | 18 ++++++------------
>>> 2 files changed, 8 insertions(+), 25 deletions(-)
>>>
>>
View attachment "dmesg_game_igt_and
xonoti_with_patch.txt" of type "text/plain" (256437 bytes)
Powered by blists - more mailing lists