[PATCH xserver/dri2 3/3] DRI2SwapBuffers: Return SBC correctly, handle target_msc correctly.

Jesse Barnes jbarnes at virtuousgeek.org
Thu Mar 4 10:20:48 PST 2010


On Sun, 21 Feb 2010 05:26:01 +0100
Mario Kleiner <mario.kleiner at tuebingen.mpg.de> wrote:

> Returns expected SBC after completion of swap to caller, as required
> by OML_sync_control spec, instead of the last_swap_target value.
> 
> Passes target_msc, divisor, remainder, correctly for
> glXSwapBuffersMscOML() call, while retaining old
> behaviour for simple glXSwapBuffers() call.
> 
> Signed-off-by: Mario Kleiner <mario.kleiner at tuebingen.mpg.de>

I think this is what it ought to look like on top of my last patch to
fix last_swap_target initialization.

Your other patches looked fine too.

diff --git a/hw/xfree86/dri2/dri2.c b/hw/xfree86/dri2/dri2.c
index 55846b1..952e493 100644
--- a/hw/xfree86/dri2/dri2.c
+++ b/hw/xfree86/dri2/dri2.c
@@ -622,11 +622,26 @@ DRI2SwapBuffers(ClientPtr client, DrawablePtr pDraw, CARD64 target_msc,
     }
 
     /*
-     * Swap target for this swap is last swap target + swap interval since
-     * we have to account for the current swap count, interval, and the
-     * number of pending swaps.
+     * In the simple glXSwapBuffers case, all params will be 0, and we just
+     * need to schedule a swap for the last swap target + the swap interval.
      */
-    *swap_target = pPriv->last_swap_target + pPriv->swap_interval;
+    if (target_msc == 0 && divisor == 0 && remainder == 0) {
+	/*
+	 * Swap target for this swap is last swap target + swap interval since
+	 * we have to account for the current swap count, interval, and the
+	 * number of pending swaps.
+	 */
+	*swap_target = pPriv->last_swap_target + pPriv->swap_interval;
+    } else {
+	/* glXSwapBuffersMscOML() call with target_msc, divisor, remainder set.
+	 * Just make sure that target_msc is not earlier than last swap target +
+	 * swap interval.
+	 */
+	if (target_msc < pPriv->last_swap_target + pPriv->swap_interval)
+	    target_msc = pPriv->last_swap_target + pPriv->swap_interval;
+
+	*swap_target = target_msc;
+    }
 
     ret = (*ds->ScheduleSwap)(client, pDraw, pDestBuffer, pSrcBuffer,
 			      swap_target, divisor, remainder, func, data);
@@ -639,6 +654,11 @@ DRI2SwapBuffers(ClientPtr client, DrawablePtr pDraw, CARD64 target_msc,
     pPriv->swapsPending++;
     pPriv->last_swap_target = *swap_target;
 
+    /* According to spec, return expected swapbuffers count SBC after this swap
+     * will complete.
+     */
+    *swap_target = pPriv->swap_count + pPriv->swapsPending;
+
     return Success;
 }
 


More information about the xorg-devel mailing list