commit a727c804a2c17db306c68e259ae845aa6382d3b1
author Thomas Hellstrom <thellstrom@vmware.com> Tue Sep 05 09:58:08 2017 +0200
committer Thomas Hellstrom <thellstrom@vmware.com> Thu Sep 07 12:43:29 2017 +0200
tree 57b09c44646aa67ae9e121fc4f05365ec371aec5
parent e4018fdd85d8b58250c2cff8cdf4edd6b2b3c70b

loader/dri3: Process event after each fence wait

This tries to mimic dri2 behaviour where events are typically processed
while waiting for X replies. Since, during steady-state dri3 rendering, we
seldom wait for xcb replies, and haven't enabled any automatic event
processing, instead check for events after a fence wait.

Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>

src/loader/loader_dri3_helper.c

diff --git a/src/loader/loader_dri3_helper.c b/src/loader/loader_dri3_helper.c
index c0a6e0d..51e4e97 100644
--- a/src/loader/loader_dri3_helper.c
+++ b/src/loader/loader_dri3_helper.c
@@ -181,10 +181,13 @@
 }
 
 static inline void
-dri3_fence_await(xcb_connection_t *c, struct loader_dri3_buffer *buffer)
+dri3_fence_await(xcb_connection_t *c, struct loader_dri3_drawable *draw,
+                 struct loader_dri3_buffer *buffer)
 {
    xcb_flush(c);
    xshmfence_await(buffer->shm_fence);
+   if (draw)
+      dri3_flush_present_events(draw);
 }
 
 static void
@@ -662,9 +665,9 @@
                      dri3_drawable_gc(draw),
                      x, y, x, y, width, height);
       dri3_fence_trigger(draw->conn, dri3_fake_front_buffer(draw));
-      dri3_fence_await(draw->conn, dri3_fake_front_buffer(draw));
+      dri3_fence_await(draw->conn, NULL, dri3_fake_front_buffer(draw));
    }
-   dri3_fence_await(draw->conn, back);
+   dri3_fence_await(draw->conn, draw, back);
 }
 
 void
@@ -680,7 +683,7 @@
                   dri3_drawable_gc(draw),
                   0, 0, 0, 0, draw->width, draw->height);
    dri3_fence_trigger(draw->conn, dri3_fake_front_buffer(draw));
-   dri3_fence_await(draw->conn, dri3_fake_front_buffer(draw));
+   dri3_fence_await(draw->conn, draw, dri3_fake_front_buffer(draw));
 }
 
 void
@@ -1374,7 +1377,7 @@
          if (buffer) {
             if (!buffer->linear_buffer) {
                dri3_fence_reset(draw->conn, new_buffer);
-               dri3_fence_await(draw->conn, buffer);
+               dri3_fence_await(draw->conn, draw, buffer);
                dri3_copy_area(draw->conn,
                               buffer->pixmap,
                               new_buffer->pixmap,
@@ -1405,7 +1408,7 @@
 
          if (new_buffer->linear_buffer &&
              draw->vtable->in_current_context(draw)) {
-            dri3_fence_await(draw->conn, new_buffer);
+            dri3_fence_await(draw->conn, draw, new_buffer);
             (void) loader_dri3_blit_image(draw,
                                           new_buffer->image,
                                           new_buffer->linear_buffer,
@@ -1417,7 +1420,7 @@
       buffer = new_buffer;
       draw->buffers[buf_id] = buffer;
    }
-   dri3_fence_await(draw->conn, buffer);
+   dri3_fence_await(draw->conn, draw, buffer);
 
    /*
     * Do we need to preserve the content of a previous buffer?