drm/amd/display: Fix deadlock during gpu reset v3
authorBhawanpreet Lakha <Bhawanpreet.Lakha@amd.com>
Mon, 11 Jan 2021 19:45:12 +0000 (14:45 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 14 Jan 2021 04:48:47 +0000 (23:48 -0500)
[Why]
during idle optimizations we acquire the dc_lock, this lock is also
acquired during gpu_reset so we end up hanging the system due to a
deadlock

[How]
If we are in gpu reset:
 - disable idle optimizations and skip calls to the dc function

v2: skip idle optimizations calls
v3: add guard for DCN

Fixes: 71338cb4a7c2b51 ("drm/amd/display: enable idle optimizations for linux (MALL stutter)")
Signed-off-by: Bhawanpreet Lakha <Bhawanpreet.Lakha@amd.com>
Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

index 2dc8493..d26296c 100644 (file)
@@ -1816,6 +1816,11 @@ static int dm_suspend(void *handle)
 
        if (amdgpu_in_reset(adev)) {
                mutex_lock(&dm->dc_lock);
+
+#if defined(CONFIG_DRM_AMD_DC_DCN)
+               dc_allow_idle_optimizations(adev->dm.dc, false);
+#endif
+
                dm->cached_dc_state = dc_copy_state(dm->dc->current_state);
 
                dm_gpureset_toggle_interrupts(adev, dm->cached_dc_state, false);
@@ -5483,6 +5488,10 @@ static inline int dm_set_vblank(struct drm_crtc *crtc, bool enable)
        if (!dc_interrupt_set(adev->dm.dc, irq_source, enable))
                return -EBUSY;
 
+#if defined(CONFIG_DRM_AMD_DC_DCN)
+       if (amdgpu_in_reset(adev))
+               return 0;
+
        mutex_lock(&dm->dc_lock);
 
        if (enable)
@@ -5499,6 +5508,7 @@ static inline int dm_set_vblank(struct drm_crtc *crtc, bool enable)
 
        mutex_unlock(&dm->dc_lock);
 
+#endif
        return 0;
 }