diff options
| author | Nirmoy Das <[email protected]> | 2022-06-24 11:08:21 +0000 |
|---|---|---|
| committer | Ramalingam C <[email protected]> | 2022-06-29 09:22:50 +0000 |
| commit | a06968563775181690125091f470a8655742dcbf (patch) | |
| tree | 304ee0b8b0e2b02d1f43ed0752f501ce9a700c01 /drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | |
| parent | drm/i915/guc/slpc: Use non-blocking H2G for waitboost (diff) | |
| download | kernel-a06968563775181690125091f470a8655742dcbf.tar.gz kernel-a06968563775181690125091f470a8655742dcbf.zip | |
drm/i915: Fix a lockdep warning at error capture
For some platfroms we use stop_machine version of
gen8_ggtt_insert_page/gen8_ggtt_insert_entries to avoid a
concurrent GGTT access bug but this causes a circular locking
dependency warning:
Possible unsafe locking scenario:
CPU0 CPU1
---- ----
lock(&ggtt->error_mutex);
lock(dma_fence_map);
lock(&ggtt->error_mutex);
lock(cpu_hotplug_lock);
Fix this by calling gen8_ggtt_insert_page/gen8_ggtt_insert_entries
directly at error capture which is concurrent GGTT access safe because
reset path make sure of that.
v2: Fix rebase conflict and added a comment.
Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/5595
Reviewed-by: Gwan-gyeong Mun <[email protected]>
Suggested-by: Chris Wilson <[email protected]>
Signed-off-by: Nirmoy Das <[email protected]>
Reviewed-by: Andrzej Hajda <[email protected]>
Signed-off-by: Ramalingam C <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_object.c')
0 files changed, 0 insertions, 0 deletions
