[PATCH] present: fix bad logic in cancelling scheduled operations.

Axel Davy axel.davy at ens.fr
Wed Jun 18 19:22:09 PDT 2014


If we present several pixmaps in advance for different msc, the later one
shouldn't cancel the previous ones.

This reverts a change made by commit
e6f5d9d7b7efdacea0f22f1808efca849bcede4c

Without this fix, vblank_mode=0 glxgears doesn't update
with the present fallback.

Signed-off-by: Axel Davy <axel.davy at ens.fr>
---
 present/present.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/present/present.c b/present/present.c
index 1bf3a58..e93851d 100644
--- a/present/present.c
+++ b/present/present.c
@@ -752,7 +752,7 @@ present_pixmap(WindowPtr window,
             if (!vblank->queued)
                 continue;
 
-            if (vblank->crtc != target_crtc || vblank->target_msc > target_msc)
+            if (vblank->crtc != target_crtc || vblank->target_msc != target_msc)
                 continue;
 
             DebugPresent(("\tx %lld %p %8lld: %08lx -> %08lx (crtc %p)\n",
-- 
1.9.1



More information about the xorg-devel mailing list