drm/amd/display: Use a reasonable timeout for framebuffer fence waits
authorAndrey Grodzovsky <andrey.grodzovsky@amd.com>
Mon, 15 Apr 2019 14:57:14 +0000 (10:57 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 23 Apr 2019 17:09:26 +0000 (12:09 -0500)
Patch '5edb0c9b Fix deadlock with display during hanged ring recovery'
was accidentaly removed during one of DALs code merges.

Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@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 36331e70fa835c17eb9b3b5780438d563b2c6cd3..a3dc7ab523deee06cdda8cc6073a935258a63025 100644 (file)
@@ -5207,11 +5207,16 @@ static void amdgpu_dm_commit_planes(struct drm_atomic_state *state,
 
                abo = gem_to_amdgpu_bo(fb->obj[0]);
 
-               /* Wait for all fences on this FB */
+               /*
+                * Wait for all fences on this FB. Do limited wait to avoid
+                * deadlock during GPU reset when this fence will not signal
+                * but we hold reservation lock for the BO.
+                */
                r = reservation_object_wait_timeout_rcu(abo->tbo.resv, true,
                                                        false,
-                                                       MAX_SCHEDULE_TIMEOUT);
-               WARN_ON(r < 0);
+                                                       msecs_to_jiffies(5000));
+               if (unlikely(r <= 0))
+                       DRM_ERROR("Waiting for fences timed out or interrupted!");
 
                /*
                 * TODO This might fail and hence better not used, wait
@@ -5220,10 +5225,8 @@ static void amdgpu_dm_commit_planes(struct drm_atomic_state *state,
                 * blocking commit to as per framework helpers
                 */
                r = amdgpu_bo_reserve(abo, true);
-               if (unlikely(r != 0)) {
+               if (unlikely(r != 0))
                        DRM_ERROR("failed to reserve buffer before flip\n");
-                       WARN_ON(1);
-               }
 
                amdgpu_bo_get_tiling_flags(abo, &tiling_flags);