path: root/present/present.c
diff options
authorChris Wilson <>2015-02-06 08:25:42 +0000
committerKeith Packard <>2015-05-26 15:09:32 -0700
commit806470b9f623089dc81b985f250f0c3a4e8edbe8 (patch)
treec9fec9593f3965991c8dfe19656ba3884dbbde9c /present/present.c
parentbaa50f60acd9e9f4293107435645ab072b6110e1 (diff)
present: Copy unflip contents back to the Screen Pixmap
As we unflip after the flip Window no longer passes the pixel ownership test for the full Screen Pixmap, we can no longer utilize that Window to copy the contents back to the backing pixmap. To first flip means that the Window was originally backed by the Screen Pixmap and wholly covered the Pixmap, thus we need to copy the last frame contents to the Screen Pixmap when the flip chain is complete. Signed-off-by: Chris Wilson <> Reviewed-and-Tested-by: Michel Dänzer <>
Diffstat (limited to 'present/present.c')
1 files changed, 4 insertions, 4 deletions
diff --git a/present/present.c b/present/present.c
index 2a705a968..a6346015e 100644
--- a/present/present.c
+++ b/present/present.c
@@ -409,20 +409,20 @@ static void
present_unflip(ScreenPtr screen)
present_screen_priv_ptr screen_priv = present_screen_priv(screen);
+ PixmapPtr pixmap = (*screen->GetScreenPixmap)(screen);
assert (!screen_priv->unflip_event_id);
assert (!screen_priv->flip_pending);
if (screen_priv->flip_window)
- present_set_tree_pixmap(screen_priv->flip_window,
- (*screen->GetScreenPixmap)(screen));
+ present_set_tree_pixmap(screen_priv->flip_window, pixmap);
- present_set_tree_pixmap(screen->root, (*screen->GetScreenPixmap)(screen));
+ present_set_tree_pixmap(screen->root, pixmap);
/* Update the screen pixmap with the current flip pixmap contents
if (screen_priv->flip_pixmap && screen_priv->flip_window) {
- present_copy_region(&screen_priv->flip_window->drawable,
+ present_copy_region(&pixmap->drawable,
NULL, 0, 0);