diff options
| author | Kristian Høgsberg <krh@bitplanet.net> | 2010-07-29 20:36:25 -0400 |
|---|---|---|
| committer | Kristian Høgsberg <krh@bitplanet.net> | 2010-08-06 08:30:47 -0400 |
| commit | 8d7b7a0d71e0b89321b3341b781bc8845386def6 (patch) | |
| tree | 023440464af3a1b8ee71b59930da17ebff001e95 | |
| parent | c65f610e12f9df168d5639534ed3c2bd40afffc8 (diff) | |
Set DamageSetReportAfterOp to true for the damage extension
Change the damage extension reporter to queue up events after we chain
to the wrapped functions. Damage events are typically sent out after
the rendering happens anyway, since we submit batch buffers from the
flush callback chain and then flush client io buffers. Compositing
managers relie on this order, and there is no way we could reliably
provide damage events to clients before the rendering happens anyway.
By queueing up the damage events before the rendering happens, there's
a risk that the client io buffer may overflow and send the damage
events to the client before the driver has even seen the rendering
request. Reporting damage events after the rendering fixes this
corner case and better corresponds with how we expect this to work.
Signed-off-by: Kristian Høgsberg <krh@bitplanet.net>
Reviewed-by: Keith Packard <keithp@keithp.com>
| -rw-r--r-- | damageext/damageext.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/damageext/damageext.c b/damageext/damageext.c index f5265ddfc..b4bb478dd 100644 --- a/damageext/damageext.c +++ b/damageext/damageext.c @@ -217,6 +217,7 @@ ProcDamageCreate (ClientPtr client) if (!AddResource (stuff->damage, DamageExtType, (pointer) pDamageExt)) return BadAlloc; + DamageSetReportAfterOp (pDamageExt->pDamage, TRUE); DamageRegister (pDamageExt->pDrawable, pDamageExt->pDamage); if (pDrawable->type == DRAWABLE_WINDOW) |