Re: [PATCH 2/2] Set DamageSetReportAfterOp to true for the damage extension

2010-11-11 Thread Michel Dänzer
On Mit, 2010-11-10 at 15:53 -0800, Keith Packard wrote: On Thu, 28 Oct 2010 20:46:23 -0700, Eric Anholt e...@anholt.net wrote: (cherry picked from commit 8d7b7a0d71e0b89321b3341b781bc8845386def6) [anholt: re-applied to revert the revert, now that the cause of the revert is fixed]

[PATCH 2/2] Set DamageSetReportAfterOp to true for the damage extension

2010-10-28 Thread Eric Anholt
From: Kristian Høgsberg k...@bitplanet.net 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

Re: [PATCH 2/2] Set DamageSetReportAfterOp to true for the damage extension

2010-10-28 Thread Keith Packard
On Thu, 28 Oct 2010 20:46:23 -0700, Eric Anholt e...@anholt.net wrote: Signed-off-by: Kristian Høgsberg k...@bitplanet.net Reviewed-by: Keith Packard kei...@keithp.com Do I need to re-review this patch? Reviewed-by: Keith Packard kei...@keithp.com (I still think the way post-op damage is