Re: [Intel-gfx] [PATCH] drm/i915: Lock dpt_obj around set_cache_level, v2.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Op 26-01-2022 om 13:55 schreef Gwan-gyeong Mun:
>
>
> On 1/26/22 9:37 AM, Maarten Lankhorst wrote:
>> set_cache_level may unbind the object, which will result in the below
>> lockdep splat:
>> <6> [184.578145] [IGT] kms_addfb_basic: starting subtest addfb25-framebuffer-vs-set-tiling
>> <4> [184.578220] ------------[ cut here ]------------
>> <4> [184.578221] WARN_ON(debug_locks && !(lock_is_held(&(&((obj)->base.resv)->lock.base)->dep_map) != 0))
>> <4> [184.578237] WARNING: CPU: 6 PID: 5544 at drivers/gpu/drm/i915/i915_gem.c:123 i915_gem_object_unbind+0x4a9/0x510 [i915]
>> <4> [184.578323] Modules linked in: vgem drm_shmem_helper snd_hda_codec_hdmi i915 mei_hdcp x86_pkg_temp_thermal snd_hda_intel coretemp crct10dif_pclmul snd_intel_dspcfg crc32_pclmul ttm snd_hda_codec ghash_clmulni_intel snd_hwdep drm_kms_helper snd_hda_core e1000e mei_me syscopyarea ptp snd_pcm sysfillrect mei pps_core sysimgblt fb_sys_fops prime_numbers intel_lpss_pci smsc75xx usbnet mii
>> <4> [184.578349] CPU: 6 PID: 5544 Comm: kms_addfb_basic Not tainted 5.16.0-CI-Patchwork_22006+ #1
>> <4> [184.578351] Hardware name: Intel Corporation Alder Lake Client Platform/AlderLake-P DDR4 RVP, BIOS ADLPFWI1.R00.2422.A00.2110131104 10/13/2021
>> <4> [184.578352] RIP: 0010:i915_gem_object_unbind+0x4a9/0x510 [i915]
>> <4> [184.578424] Code: 00 be ff ff ff ff 48 8d 78 68 e8 a2 6e 2b e1 85 c0 0f 85 b1 fb ff ff 48 c7 c6 48 37 9e a0 48 c7 c7 d9 fc a1 a0 e8 a3 54 26 e1 <0f> 0b e9 97 fb ff ff 31 ed 48 8b 5c 24 58 65 48 33 1c 25 28 00 00
>> <4> [184.578426] RSP: 0018:ffffc900013b3b68 EFLAGS: 00010286
>> <4> [184.578428] RAX: 0000000000000000 RBX: ffffc900013b3bb0 RCX: 0000000000000001
>> <4> [184.578429] RDX: 0000000080000001 RSI: ffffffff8230b42d RDI: 00000000ffffffff
>> <4> [184.578430] RBP: ffff888120e10000 R08: 0000000000000000 R09: c0000000ffff7fff
>> <4> [184.578431] R10: 0000000000000001 R11: ffffc900013b3980 R12: ffff8881176ea740
>> <4> [184.578432] R13: ffff888120e10000 R14: 0000000000000000 R15: 0000000000000001
>> <4> [184.578433] FS:  00007f65074f5e40(0000) GS:ffff88888f300000(0000) knlGS:0000000000000000
>> <4> [184.578435] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>> <4> [184.578436] CR2: 00007fff4420ede8 CR3: 000000010c2f2005 CR4: 0000000000770ee0
>> <4> [184.578437] PKRU: 55555554
>> <4> [184.578438] Call Trace:
>> <4> [184.578439]  <TASK>
>> <4> [184.578440]  ? dma_resv_iter_first_unlocked+0x78/0xf0
>> <4> [184.578447]  intel_dpt_create+0x88/0x220 [i915]
>> <4> [184.578530]  intel_framebuffer_init+0x5b8/0x620 [i915]
>> <4> [184.578612]  intel_framebuffer_create+0x3d/0x60 [i915]
>> <4> [184.578691]  intel_user_framebuffer_create+0x18f/0x2c0 [i915]
>> <4> [184.578775]  drm_internal_framebuffer_create+0x36d/0x4c0
>> <4> [184.578779]  drm_mode_addfb2+0x2f/0xd0
>> <4> [184.578781]  ? drm_mode_addfb_ioctl+0x10/0x10
>> <4> [184.578784]  drm_ioctl_kernel+0xac/0x140
>> <4> [184.578787]  drm_ioctl+0x201/0x3d0
>> <4> [184.578789]  ? drm_mode_addfb_ioctl+0x10/0x10
>> <4> [184.578796]  __x64_sys_ioctl+0x6a/0xa0
>> <4> [184.578800]  do_syscall_64+0x37/0xb0
>> <4> [184.578803]  entry_SYSCALL_64_after_hwframe+0x44/0xae
>> <4> [184.578805] RIP: 0033:0x7f6506736317
>> <4> [184.578807] Code: b3 66 90 48 8b 05 71 4b 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 41 4b 2d 00 f7 d8 64 89 01 48
>> <4> [184.578808] RSP: 002b:00007fff44211a98 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
>> <4> [184.578810] RAX: ffffffffffffffda RBX: 0000000000000006 RCX: 00007f6506736317
>> <4> [184.578811] RDX: 00007fff44211b30 RSI: 00000000c06864b8 RDI: 0000000000000006
>> <4> [184.578812] RBP: 00007fff44211b30 R08: 00007fff44311170 R09: 0000000000000000
>> <4> [184.578813] R10: 0000000000000008 R11: 0000000000000246 R12: 00000000c06864b8
>> <4> [184.578813] R13: 0000000000000006 R14: 0000000000000000 R15: 0000000000000000
>> <4> [184.578819]  </TASK>
>> <4> [184.578820] irq event stamp: 47931
>> <4> [184.578821] hardirqs last  enabled at (47937): [<ffffffff81130dd2>] __up_console_sem+0x62/0x70
>> <4> [184.578824] hardirqs last disabled at (47942): [<ffffffff81130db7>] __up_console_sem+0x47/0x70
>> <4> [184.578826] softirqs last  enabled at (47340): [<ffffffff81e0032d>] __do_softirq+0x32d/0x493
>> <4> [184.578828] softirqs last disabled at (47335): [<ffffffff810b9196>] irq_exit_rcu+0xa6/0xe0
>> <4> [184.578830] ---[ end trace f17ec219f892c7d4 ]---
>>
>> Changes since v1:
>> - Fix intel_pin_fb_obj_dpt too.
>>
>> Fixes: 0f341974cbc2 ("drm/i915: Add i915_vma_unbind_unlocked, and take obj lock for i915_vma_unbind, v2.")
>> Signed-off-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
>> Testcase: kms_addfb_basic
>> ---
>>   drivers/gpu/drm/i915/display/intel_dpt.c    | 6 +++++-
>>   drivers/gpu/drm/i915/display/intel_fb_pin.c | 6 +++++-
>>   2 files changed, 10 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_dpt.c b/drivers/gpu/drm/i915/display/intel_dpt.c
>> index 63a83d5f85a1..c2f8f853db90 100644
>> --- a/drivers/gpu/drm/i915/display/intel_dpt.c
>> +++ b/drivers/gpu/drm/i915/display/intel_dpt.c
>> @@ -253,7 +253,11 @@ intel_dpt_create(struct intel_framebuffer *fb)
>>       if (IS_ERR(dpt_obj))
>>           return ERR_CAST(dpt_obj);
>>   -    ret = i915_gem_object_set_cache_level(dpt_obj, I915_CACHE_NONE);
>> +    ret = i915_gem_object_lock_interruptible(dpt_obj, NULL);
>> +    if (!ret) {
>> +        ret = i915_gem_object_set_cache_level(dpt_obj, I915_CACHE_NONE);
>> +        i915_gem_object_unlock(dpt_obj);
>> +    }
>>       if (ret) {
>>           i915_gem_object_put(dpt_obj);
>>           return ERR_PTR(ret);
>> diff --git a/drivers/gpu/drm/i915/display/intel_fb_pin.c b/drivers/gpu/drm/i915/display/intel_fb_pin.c
>> index 9c555f6d1958..c4b3d76341f3 100644
>> --- a/drivers/gpu/drm/i915/display/intel_fb_pin.c
>> +++ b/drivers/gpu/drm/i915/display/intel_fb_pin.c
>> @@ -36,7 +36,11 @@ intel_pin_fb_obj_dpt(struct drm_framebuffer *fb,
>>         atomic_inc(&dev_priv->gpu_error.pending_fb_pin);
>>   -    ret = i915_gem_object_set_cache_level(obj, I915_CACHE_NONE);
>> +    ret = i915_gem_object_lock_interruptible(obj, NULL);
>> +    if (!ret) {
>> +        ret = i915_gem_object_set_cache_level(obj, I915_CACHE_NONE);
>> +        i915_gem_object_unlock(obj);
>> +    }
>>       if (ret) {
>>           vma = ERR_PTR(ret);
>>           goto err;
>>
>
> Hi Maarten,
>
> Even if v2 is applied, when the igt_runner runs with fast-feedback.testlist as input,lock problem log occurs in dmesg below in kms_busy test.
>
> Testing was done in adlp.
>
> I think the lock check routine would have to be added even when doing intel_dpt_destroy(), what do you think?
>
>
> -----------------------------------------------------------------------------------------------------------
> [  374.868359] ------------[ cut here ]------------
> [  374.868409] WARNING: CPU: 2 PID: 6696 at drivers/gpu/drm/i915/gem/i915_gem_object.h:171 __i915_vma_unbind+0xdf/0xf0 [i915]
> [  374.868506] Modules linked in: vgem drm_shmem_helper fuse snd_hda_codec_hdmi smsc75xx usbnet mii mei_hdcp x86_pkg_temp_thermal snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep coretemp snd_hda_core snd_pcm mei_me mei i915 drm_buddy crct10dif_pclmul crc32_pclmul ttm ghash_clmulni_intel drm_dp_helper i2c_i801 drm_kms_helper i2c_smbus intel_lpss_pci syscopyarea sysfillrect sysimgblt fb_sys_fops prime_numbers
> [  374.868539] CPU: 2 PID: 6696 Comm: kms_busy Not tainted 5.17.0-rc1-drm-tip-test+ #26
> [  374.868541] Hardware name: Intel Corporation Alder Lake Client Platform/AlderLake-P DDR5 RVP, BIOS ADLPFWI1.R00.2162.A00.2104190450 04/19/2021
> [  374.868543] RIP: 0010:__i915_vma_unbind+0xdf/0xf0 [i915]
> [  374.868631] Code: 15 32 1c 5c e2 85 d2 0f 84 5c ff ff ff 48 8b b8 38 01 00 00 be ff ff ff ff 48 83 c7 68 e8 a9 e0 8a e1 85 c0 0f 85 3f ff ff ff <0f> 0b e9 38 ff ff ff 41 bc f5 ff ff ff eb 86 66 90 0f 1f 44 00 00
> [  374.868633] RSP: 0018:ffffc900022f3cd0 EFLAGS: 00010246
> [  374.868636] RAX: 0000000000000000 RBX: ffff88810c204b40 RCX: 0000000000000000
> [  374.868637] RDX: 0000000080000000 RSI: ffffffff822dc04f RDI: ffffffff82339ebf
> [  374.868638] RBP: ffff88810c204b40 R08: 00000000ffffffff R09: 0000000000000000
> [  374.868639] R10: 0000000000000001 R11: ffffffffa023ba80 R12: ffff8881095ebb18
> [  374.868641] R13: ffff8881095eb800 R14: 00000000ffffffff R15: ffff8881095eb848
> [  374.868642] FS:  00007f0326c43c00(0000) GS:ffff88849f300000(0000) knlGS:0000000000000000
> [  374.868643] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [  374.868645] CR2: 00007f0329403938 CR3: 00000001429c4006 CR4: 0000000000770ee0
> [  374.868646] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [  374.868647] DR3: 0000000000000000 DR6: 00000000ffff07f0 DR7: 0000000000000400
> [  374.868649] PKRU: 55555554
> [  374.868650] Call Trace:
> [  374.868651]  <TASK>
> [  374.868654]  __i915_vm_close+0xab/0x190 [i915]
> [  374.868729]  intel_dpt_destroy+0x29/0x60 [i915]
> [  374.868823]  intel_user_framebuffer_destroy+0x48/0x50 [i915]
> [  374.868945]  drm_mode_rmfb+0x1eb/0x220
> [  374.868958]  ? drm_mode_rmfb+0x220/0x220
> [  374.868963]  ? drm_mode_rmfb+0x220/0x220
> [  374.868966]  drm_ioctl_kernel+0xb1/0x140
> [  374.868973]  drm_ioctl+0x201/0x3b0
> [  374.868977]  ? drm_mode_rmfb+0x220/0x220
> [  374.868981]  ? tty_ioctl+0x447/0x980
> [  374.868986]  ? find_held_lock+0x2b/0x80
> [  374.868995]  __x64_sys_ioctl+0x83/0xa0
> [  374.869000]  do_syscall_64+0x38/0xc0
> [  374.869006]  entry_SYSCALL_64_after_hwframe+0x44/0xae
> [  374.869009] RIP: 0033:0x7f0328fde50b
> [  374.869012] Code: 0f 1e fa 48 8b 05 85 39 0d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 0f 1f 44 00 00 f3 0f 1e fa b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 55 39 0d 00 f7 d8 64 89 01 48
> [  374.869014] RSP: 002b:00007ffec4a3f278 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
> [  374.869017] RAX: ffffffffffffffda RBX: 00007ffec4a3f2bc RCX: 00007f0328fde50b
> [  374.869019] RDX: 00007ffec4a3f2bc RSI: 00000000c00464af RDI: 0000000000000005
> [  374.869020] RBP: 00000000c00464af R08: 0000000000000000 R09: 0000000000000000
> [  374.869021] R10: fffffffffffffdea R11: 0000000000000246 R12: 00007ffec4a3f360
> [  374.869022] R13: 0000000000000005 R14: 00007ffec4a3f400 R15: 0000000000000000
> [  374.869032]  </TASK>
> [  374.869034] irq event stamp: 258641
> [  374.869035] hardirqs last  enabled at (258647): [<ffffffff8113a437>] __up_console_sem+0x67/0x70
> [  374.869039] hardirqs last disabled at (258652): [<ffffffff8113a41c>] __up_console_sem+0x4c/0x70
> [  374.869041] softirqs last  enabled at (257220): [<ffffffff810c104d>] irq_exit_rcu+0xdd/0x130
> [  374.869045] softirqs last disabled at (257215): [<ffffffff810c104d>] irq_exit_rcu+0xdd/0x130
> [  374.869047] ---[ end trace 0000000000000000 ]---
> [  374.869178] ------------[ cut here ]------------
> [  374.869180] WARNING: CPU: 2 PID: 6696 at drivers/gpu/drm/i915/gem/i915_gem_object.h:171 __i915_vma_evict+0x2b0/0x330 [i915]
> [  374.869272] Modules linked in: vgem drm_shmem_helper fuse snd_hda_codec_hdmi smsc75xx usbnet mii mei_hdcp x86_pkg_temp_thermal snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep coretemp snd_hda_core snd_pcm mei_me mei i915 drm_buddy crct10dif_pclmul crc32_pclmul ttm ghash_clmulni_intel drm_dp_helper i2c_i801 drm_kms_helper i2c_smbus intel_lpss_pci syscopyarea sysfillrect sysimgblt fb_sys_fops prime_numbers
> [  374.869303] CPU: 2 PID: 6696 Comm: kms_busy Tainted: G        W    5.17.0-rc1-drm-tip-test+ #26
> [  374.869305] Hardware name: Intel Corporation Alder Lake Client Platform/AlderLake-P DDR5 RVP, BIOS ADLPFWI1.R00.2162.A00.2104190450 04/19/2021
> [  374.869306] RIP: 0010:__i915_vma_evict+0x2b0/0x330 [i915]
> [  374.869389] Code: 00 00 00 e8 d2 3c 2e e1 e9 84 fe ff ff 48 8b b8 38 01 00 00 be ff ff ff ff 48 83 c7 68 e8 08 e2 8a e1 85 c0 0f 85 91 fd ff ff <0f> 0b e9 8a fd ff ff 0f 0b e9 ca fd ff ff e8 2d e3 8a e1 85 c0 0f
> [  374.869391] RSP: 0018:ffffc900022f3cb0 EFLAGS: 00010246
> [  374.869393] RAX: 0000000000000000 RBX: 0000000000000000 RCX: 0000000000000000
> [  374.869394] RDX: 0000000080000000 RSI: ffffffff822dc04f RDI: ffffffff82339ebf
> [  374.869395] RBP: ffff88810c204b40 R08: 00000000ffffffff R09: 0000000000000001
> [  374.869396] R10: 0000000000000001 R11: ffff88810156ddc8 R12: ffff88813fcc34c0
> [  374.869397] R13: ffff8881095eb800 R14: 00000000ffffffff R15: ffff8881095eb848
> [  374.869398] FS:  00007f0326c43c00(0000) GS:ffff88849f300000(0000) knlGS:0000000000000000
> [  374.869400] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [  374.869401] CR2: 00007f0329403938 CR3: 00000001429c4006 CR4: 0000000000770ee0
> [  374.869402] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [  374.869403] DR3: 0000000000000000 DR6: 00000000ffff07f0 DR7: 0000000000000400
> [  374.869404] PKRU: 55555554
> [  374.869405] Call Trace:
> [  374.869406]  <TASK>
> [  374.869408]  __i915_vma_unbind+0x6c/0xf0 [i915]
> [  374.869491]  __i915_vm_close+0xab/0x190 [i915]
> [  374.869565]  intel_dpt_destroy+0x29/0x60 [i915]
> [  374.869658]  intel_user_framebuffer_destroy+0x48/0x50 [i915]
> [  374.869750]  drm_mode_rmfb+0x1eb/0x220
> [  374.869757]  ? drm_mode_rmfb+0x220/0x220
> [  374.869761]  ? drm_mode_rmfb+0x220/0x220
> [  374.869765]  drm_ioctl_kernel+0xb1/0x140
> [  374.869771]  drm_ioctl+0x201/0x3b0
> [  374.869774]  ? drm_mode_rmfb+0x220/0x220
> [  374.869779]  ? tty_ioctl+0x447/0x980
> [  374.869782]  ? find_held_lock+0x2b/0x80
> [  374.869789]  __x64_sys_ioctl+0x83/0xa0
> [  374.869793]  do_syscall_64+0x38/0xc0
> [  374.869797]  entry_SYSCALL_64_after_hwframe+0x44/0xae
> [  374.869800] RIP: 0033:0x7f0328fde50b
> [  374.869802] Code: 0f 1e fa 48 8b 05 85 39 0d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 0f 1f 44 00 00 f3 0f 1e fa b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 55 39 0d 00 f7 d8 64 89 01 48
> [  374.869803] RSP: 002b:00007ffec4a3f278 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
> [  374.869805] RAX: ffffffffffffffda RBX: 00007ffec4a3f2bc RCX: 00007f0328fde50b
> [  374.869807] RDX: 00007ffec4a3f2bc RSI: 00000000c00464af RDI: 0000000000000005
> [  374.869808] RBP: 00000000c00464af R08: 0000000000000000 R09: 0000000000000000
> [  374.869809] R10: fffffffffffffdea R11: 0000000000000246 R12: 00007ffec4a3f360
> [  374.869810] R13: 0000000000000005 R14: 00007ffec4a3f400 R15: 0000000000000000
> [  374.869819]  </TASK>
> [  374.869820] irq event stamp: 259173
> [  374.869821] hardirqs last  enabled at (259179): [<ffffffff8113a437>] __up_console_sem+0x67/0x70
> [  374.869824] hardirqs last disabled at (259184): [<ffffffff8113a41c>] __up_console_sem+0x4c/0x70
> [  374.869827] softirqs last  enabled at (257220): [<ffffffff810c104d>] irq_exit_rcu+0xdd/0x130
> [  374.869841] softirqs last disabled at (259195): [<ffffffff810c104d>] irq_exit_rcu+0xdd/0x130
> [  374.869843] ---[ end trace 0000000000000000 ]---
> [  374.892030] Console: switching to colour frame buffer device 240x67
> [  374.920158] ------------[ cut here ]------------
> [  374.920161] WARNING: CPU: 2 PID: 6696 at drivers/gpu/drm/i915/gem/i915_gem_object.h:171 __i915_vma_unbind+0xdf/0xf0 [i915]
> [  374.920298] Modules linked in: vgem drm_shmem_helper fuse snd_hda_codec_hdmi smsc75xx usbnet mii mei_hdcp x86_pkg_temp_thermal snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep coretemp snd_hda_core snd_pcm mei_me mei i915 drm_buddy crct10dif_pclmul crc32_pclmul ttm ghash_clmulni_intel drm_dp_helper i2c_i801 drm_kms_helper i2c_smbus intel_lpss_pci syscopyarea sysfillrect sysimgblt fb_sys_fops prime_numbers
> [  374.920327] CPU: 2 PID: 6696 Comm: kms_busy Tainted: G        W    5.17.0-rc1-drm-tip-test+ #26
> [  374.920329] Hardware name: Intel Corporation Alder Lake Client Platform/AlderLake-P DDR5 RVP, BIOS ADLPFWI1.R00.2162.A00.2104190450 04/19/2021
> [  374.920331] RIP: 0010:__i915_vma_unbind+0xdf/0xf0 [i915]
> [  374.920418] Code: 15 32 1c 5c e2 85 d2 0f 84 5c ff ff ff 48 8b b8 38 01 00 00 be ff ff ff ff 48 83 c7 68 e8 a9 e0 8a e1 85 c0 0f 85 3f ff ff ff <0f> 0b e9 38 ff ff ff 41 bc f5 ff ff ff eb 86 66 90 0f 1f 44 00 00
> [  374.920420] RSP: 0018:ffffc900022f3ba0 EFLAGS: 00010246
> [  374.920422] RAX: 0000000000000000 RBX: ffff88810c205f40 RCX: 0000000000000000
> [  374.920424] RDX: 0000000080000000 RSI: ffffffff822dc04f RDI: ffffffff82339ebf
> [  374.920425] RBP: ffff88810c205f40 R08: 00000000ffffffff R09: 0000000000000000
> [  374.920426] R10: 0000000000000001 R11: ffffffffa023ba80 R12: ffff888140ecd318
> [  374.920427] R13: ffff888140ecd000 R14: 00000000ffffffff R15: ffff888140ecd048
> [  374.920428] FS:  0000000000000000(0000) GS:ffff88849f300000(0000) knlGS:0000000000000000
> [  374.920430] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [  374.920431] CR2: 00007f032940af28 CR3: 0000000006612001 CR4: 0000000000770ee0
> [  374.920432] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [  374.920433] DR3: 0000000000000000 DR6: 00000000ffff07f0 DR7: 0000000000000400
> [  374.920435] PKRU: 55555554
> [  374.920436] Call Trace:
> [  374.920437]  <TASK>
> [  374.920439]  __i915_vm_close+0xab/0x190 [i915]
> [  374.920513]  intel_dpt_destroy+0x29/0x60 [i915]
> [  374.920608]  intel_user_framebuffer_destroy+0x48/0x50 [i915]
> [  374.920697]  drm_fb_release+0x108/0x1c0
> [  374.920702]  ? find_held_lock+0x2b/0x80
> [  374.920710]  drm_file_free.part.0+0x1a4/0x260
> [  374.920716]  drm_release_noglobal+0x1c/0xa0
> [  374.920719]  __fput+0xa3/0x260
> [  374.920724]  task_work_run+0x59/0xa0
> [  374.920729]  do_exit+0x382/0xc00
> [  374.920735]  do_group_exit+0x33/0xb0
> [  374.920737]  get_signal+0x15e/0xe00
> [  374.920744]  arch_do_signal_or_restart+0xf0/0x7f0
> [  374.920754]  exit_to_user_mode_prepare+0x148/0x1d0
> [  374.920758]  syscall_exit_to_user_mode+0x19/0x50
> [  374.920762]  do_syscall_64+0x48/0xc0
> [  374.920765]  entry_SYSCALL_64_after_hwframe+0x44/0xae
> [  374.920768] RIP: 0033:0x7f0328fde50b
> [  374.920769] Code: Unable to access opcode bytes at RIP 0x7f0328fde4e1.
> [  374.920770] RSP: 002b:00007ffec4a3f278 EFLAGS: 00000246
> [  374.920772] RAX: 0000000000000000 RBX: 00007ffec4a3f2bc RCX: 00007f0328fde50b
> [  374.920774] RDX: 00007ffec4a3f2bc RSI: 00000000c00464af RDI: 0000000000000005
> [  374.920775] RBP: 00000000c00464af R08: 0000000000000000 R09: 0000000000000000
> [  374.920776] R10: fffffffffffffdea R11: 0000000000000246 R12: 00007ffec4a3f360
> [  374.920777] R13: 0000000000000005 R14: 00007ffec4a3f400 R15: 0000000000000000
> [  374.920784]  </TASK>
> [  374.920785] irq event stamp: 365965
> [  374.920786] hardirqs last  enabled at (365971): [<ffffffff8113a437>] __up_console_sem+0x67/0x70
> [  374.920789] hardirqs last disabled at (365976): [<ffffffff8113a41c>] __up_console_sem+0x4c/0x70
> [  374.920792] softirqs last  enabled at (365380): [<ffffffff81a8c945>] unix_release_sock+0xf5/0x4f0
> [  374.920795] softirqs last disabled at (365378): [<ffffffff81a8c91e>] unix_release_sock+0xce/0x4f0
> [  374.920796] ---[ end trace 0000000000000000 ]---
> [  374.920813] ------------[ cut here ]------------
> [  374.920814] WARNING: CPU: 2 PID: 6696 at drivers/gpu/drm/i915/gem/i915_gem_object.h:171 __i915_vma_evict+0x2b0/0x330 [i915]
> [  374.920972] Modules linked in: vgem drm_shmem_helper fuse snd_hda_codec_hdmi smsc75xx usbnet mii mei_hdcp x86_pkg_temp_thermal snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep coretemp snd_hda_core snd_pcm mei_me mei i915 drm_buddy crct10dif_pclmul crc32_pclmul ttm ghash_clmulni_intel drm_dp_helper i2c_i801 drm_kms_helper i2c_smbus intel_lpss_pci syscopyarea sysfillrect sysimgblt fb_sys_fops prime_numbers
> [  374.921003] CPU: 2 PID: 6696 Comm: kms_busy Tainted: G        W    5.17.0-rc1-drm-tip-test+ #26
> [  374.921005] Hardware name: Intel Corporation Alder Lake Client Platform/AlderLake-P DDR5 RVP, BIOS ADLPFWI1.R00.2162.A00.2104190450 04/19/2021
> [  374.921006] RIP: 0010:__i915_vma_evict+0x2b0/0x330 [i915]
> [  374.921095] Code: 00 00 00 e8 d2 3c 2e e1 e9 84 fe ff ff 48 8b b8 38 01 00 00 be ff ff ff ff 48 83 c7 68 e8 08 e2 8a e1 85 c0 0f 85 91 fd ff ff <0f> 0b e9 8a fd ff ff 0f 0b e9 ca fd ff ff e8 2d e3 8a e1 85 c0 0f
> [  374.921097] RSP: 0018:ffffc900022f3b80 EFLAGS: 00010246
> [  374.921102] RAX: 0000000000000000 RBX: 0000000000000000 RCX: 0000000000000000
> [  374.921103] RDX: 0000000080000000 RSI: ffffffff822dc04f RDI: ffffffff82339ebf
> [  374.921104] RBP: ffff88810c205f40 R08: 00000000ffffffff R09: 0000000000000001
> [  374.921105] R10: 0000000000000001 R11: ffff88810156ddc8 R12: ffff88813fcc6940
> [  374.921107] R13: ffff888140ecd000 R14: 00000000ffffffff R15: ffff888140ecd048
> [  374.921108] FS:  0000000000000000(0000) GS:ffff88849f300000(0000) knlGS:0000000000000000
> [  374.921109] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [  374.921111] CR2: 00007f032940af28 CR3: 0000000006612001 CR4: 0000000000770ee0
> [  374.921112] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [  374.921113] DR3: 0000000000000000 DR6: 00000000ffff07f0 DR7: 0000000000000400
> [  374.921114] PKRU: 55555554
> [  374.921115] Call Trace:
> [  374.921116]  <TASK>
> [  374.921118]  __i915_vma_unbind+0x6c/0xf0 [i915]
> [  374.921207]  __i915_vm_close+0xab/0x190 [i915]
> [  374.921284]  intel_dpt_destroy+0x29/0x60 [i915]
> [  374.921377]  intel_user_framebuffer_destroy+0x48/0x50 [i915]
> [  374.921469]  drm_fb_release+0x108/0x1c0
> [  374.921472]  ? find_held_lock+0x2b/0x80
> [  374.921480]  drm_file_free.part.0+0x1a4/0x260
> [  374.921485]  drm_release_noglobal+0x1c/0xa0
> [  374.921487]  __fput+0xa3/0x260
> [  374.921491]  task_work_run+0x59/0xa0
> [  374.921495]  do_exit+0x382/0xc00
> [  374.921500]  do_group_exit+0x33/0xb0
> [  374.921503]  get_signal+0x15e/0xe00
> [  374.921510]  arch_do_signal_or_restart+0xf0/0x7f0
> [  374.921519]  exit_to_user_mode_prepare+0x148/0x1d0
> [  374.921522]  syscall_exit_to_user_mode+0x19/0x50
> [  374.921525]  do_syscall_64+0x48/0xc0
> [  374.921528]  entry_SYSCALL_64_after_hwframe+0x44/0xae
> [  374.921531] RIP: 0033:0x7f0328fde50b
> [  374.921532] Code: Unable to access opcode bytes at RIP 0x7f0328fde4e1.
> [  374.921533] RSP: 002b:00007ffec4a3f278 EFLAGS: 00000246
> [  374.921535] RAX: 0000000000000000 RBX: 00007ffec4a3f2bc RCX: 00007f0328fde50b
> [  374.921536] RDX: 00007ffec4a3f2bc RSI: 00000000c00464af RDI: 0000000000000005
> [  374.921538] RBP: 00000000c00464af R08: 0000000000000000 R09: 0000000000000000
> [  374.921539] R10: fffffffffffffdea R11: 0000000000000246 R12: 00007ffec4a3f360
> [  374.921540] R13: 0000000000000005 R14: 00007ffec4a3f400 R15: 0000000000000000
> [  374.921547]  </TASK>
> [  374.921548] irq event stamp: 366515
> [  374.921549] hardirqs last  enabled at (366521): [<ffffffff8113a437>] __up_console_sem+0x67/0x70
> [  374.921552] hardirqs last disabled at (366526): [<ffffffff8113a41c>] __up_console_sem+0x4c/0x70
> [  374.921555] softirqs last  enabled at (365380): [<ffffffff81a8c945>] unix_release_sock+0xf5/0x4f0
> [  374.921556] softirqs last disabled at (365378): [<ffffffff81a8c91e>] unix_release_sock+0xce/0x4f0
> [  374.921558] ---[ end trace 0000000000000000 ]---

The bug on vm_close is a separate bug, and would probably best be fixed in a separate patch.

Could I get a r-b on this? It should fix some issues, even if the unbind there is a separate bug.

~Maarten




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux