Mesa (10.3): Revert "glx/dri3: Request non-vsynced Present for swapinterval zero. (v3)"

[Emil Velikov]

Module: Mesa
Branch: 10.3
Commit: 78a134c04ede497bb21bbab3127365bbd8eca7d9
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=78a134c04ede497bb21bbab3127365bbd8eca7d9

Author: Emil Velikov <emil.l.velikov at gmail.com>
Date:   Tue Dec 30 00:56:33 2014 +0000

Revert "glx/dri3: Request non-vsynced Present for swapinterval zero. (v3)"

This reverts commit d332e5046e3b8cd92a819f2d9c9ca976f0f4eb0c.

May not be the correct fix. Discussion is ongoing.

http://lists.freedesktop.org/archives/mesa-dev/2014-December/072969.html

---

 src/glx/dri3_glx.c |   14 +-------------
 1 file changed, 1 insertion(+), 13 deletions(-)

diff --git a/src/glx/dri3_glx.c b/src/glx/dri3_glx.c
index c542df3..16533a7 100644
--- a/src/glx/dri3_glx.c
+++ b/src/glx/dri3_glx.c
@@ -1483,7 +1483,6 @@ dri3_swap_buffers(__GLXDRIdrawable *pdraw, int64_t target_msc, int64_t divisor,
    xcb_connection_t *c = XGetXCBConnection(dpy);
    struct dri3_buffer *back;
    int64_t ret = 0;
-   uint32_t options = XCB_PRESENT_OPTION_NONE;
 
    unsigned flags = __DRI2_FLUSH_DRAWABLE;
    if (flush)
@@ -1536,17 +1535,6 @@ dri3_swap_buffers(__GLXDRIdrawable *pdraw, int64_t target_msc, int64_t divisor,
          remainder = 0;
       }
 
-      /* From the GLX_EXT_swap_control spec:
-       *
-       *     "If <interval> is set to a value of 0, buffer swaps are not
-       *      synchronized to a video frame."
-       *
-       * Implementation note: It is possible to enable triple buffering behaviour
-       * by not using XCB_PRESENT_OPTION_ASYNC, but this should not be the default.
-       */
-      if (priv->swap_interval == 0)
-          options |= XCB_PRESENT_OPTION_ASYNC;
-
       back->busy = 1;
       back->last_swap = priv->send_sbc;
       xcb_present_pixmap(c,
@@ -1560,7 +1548,7 @@ dri3_swap_buffers(__GLXDRIdrawable *pdraw, int64_t target_msc, int64_t divisor,
                          None,                                 /* target_crtc */
                          None,
                          back->sync_fence,
-                         options,
+                         XCB_PRESENT_OPTION_NONE,
                          target_msc,
                          divisor,
                          remainder, 0, NULL);