mirror of
https://gitlab.freedesktop.org/xorg/xserver.git
synced 2025-12-26 21:40:05 +01:00
DRI2: fix swapbuffers handling of SBC and target MSC
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.
An OML swap can have a 0 target_msc, which just means it needs to
satisfy the divisor/remainder equation. Pass this down to the driver as
needed so we can support it.
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
(cherry picked from commit b180e43977)
This commit is contained in:
parent
a56178402e
commit
23d78005b0
1 changed files with 18 additions and 4 deletions
|
|
@ -632,11 +632,20 @@ 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 could have a 0 target_msc, honor it */
|
||||
*swap_target = target_msc;
|
||||
}
|
||||
|
||||
ret = (*ds->ScheduleSwap)(client, pDraw, pDestBuffer, pSrcBuffer,
|
||||
swap_target, divisor, remainder, func, data);
|
||||
|
|
@ -649,6 +658,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;
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue