Can you try this patch ? I noticed there is a new API to use to wait for flips instead of what we are using now.

Thanks,
Andrey

On 2017-11-14 01:06 PM, Michel Dänzer wrote:
I sometimes get the attached dmesg splat when the GDM login screen times
out and fades to black (presumably specifically when it finally sets
DPMS off, which disables the vblank interrupt).

I've seen this for the first time today, so it might have been
introduced by one of the DC changes which landed in amd-staging-drm-next
since yesterday. But since it doesn't happen every time, I can't be sure
of that yet.



>From 301759d8cb7a4ccfbe3a219d91bebbec9a256d11 Mon Sep 17 00:00:00 2001
From: Andrey Grodzovsky <[email protected]>
Date: Tue, 14 Nov 2017 20:45:52 -0500
Subject: drm/amdgpu/display: Switch to drm_atomic_helper_wait_for_flip_done

This new helper function is advised to be used for drviers that
use the nonblocking commit tracking support instead of
drm_atomic_helper_wait_for_vblanks.

Signed-off-by: Andrey Grodzovsky <[email protected]>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index 39e751d..088d154 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -4255,7 +4255,7 @@ static void amdgpu_dm_atomic_commit_tail(struct drm_atomic_state *state)
 	drm_atomic_helper_commit_hw_done(state);
 
 	if (wait_for_vblank)
-		drm_atomic_helper_wait_for_vblanks(dev, state);
+		drm_atomic_helper_wait_for_flip_done(dev, state);
 
 	drm_atomic_helper_cleanup_planes(dev, state);
 }
-- 
2.7.4

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to