Re: [PATCH xserver] damage: Add screen func called before damage event delivery

2016-09-20 Thread Michel Dänzer
On 17/09/16 12:51 AM, Keith Packard wrote: > Michel Dänzer writes: > >> Does FlushClient get called after every DamageExtNotify call? Otherwise, >> some of the GPU flushes performed by DamageFlushDrawable will be wasted, >> hurting performance. > > The driver gets told which

Re: [PATCH xserver] damage: Add screen func called before damage event delivery

2016-09-16 Thread Keith Packard
Michel Dänzer writes: > I'm afraid I'm not sure this is going in a good direction. Yeah, with the WriteToClient changes from a few months ago, I think we need to do something though. We used to be able to rely on no events being delivered before the BlockHandler was invoked.

Re: [PATCH xserver] damage: Add screen func called before damage event delivery

2016-09-16 Thread Michel Dänzer
On 16/09/16 06:14 PM, Hans de Goede wrote: > On 16-09-16 10:59, Michel Dänzer wrote: >> On 16/09/16 05:37 PM, Hans de Goede wrote: >>> On 16-09-16 08:26, Michel Dänzer wrote: On 16/09/16 01:37 PM, Keith Packard wrote: > @@ -1943,3 +1948,13 @@ DamageReportDamage(DamagePtr pDamage, >

Re: [PATCH xserver] damage: Add screen func called before damage event delivery

2016-09-16 Thread Hans de Goede
Hi, On 16-09-16 10:59, Michel Dänzer wrote: On 16/09/16 05:37 PM, Hans de Goede wrote: On 16-09-16 08:26, Michel Dänzer wrote: On 16/09/16 01:37 PM, Keith Packard wrote: This lets the video driver flush rendering to the kernel before the client receives a damage event to a pixmap which the

Re: [PATCH xserver] damage: Add screen func called before damage event delivery

2016-09-16 Thread Michel Dänzer
On 16/09/16 05:37 PM, Hans de Goede wrote: > On 16-09-16 08:26, Michel Dänzer wrote: >> On 16/09/16 01:37 PM, Keith Packard wrote: >>> This lets the video driver flush rendering to the kernel before the >>> client receives a damage event to a pixmap which the client has direct >>> rendering access

Re: [PATCH xserver] damage: Add screen func called before damage event delivery

2016-09-16 Thread Hans de Goede
Hi, On 16-09-16 08:26, Michel Dänzer wrote: On 16/09/16 01:37 PM, Keith Packard wrote: This lets the video driver flush rendering to the kernel before the client receives a damage event to a pixmap which the client has direct rendering access to. I'm afraid I'm not sure this is going in a

Re: [PATCH xserver] damage: Add screen func called before damage event delivery

2016-09-16 Thread Michel Dänzer
On 16/09/16 01:37 PM, Keith Packard wrote: > This lets the video driver flush rendering to the kernel before the > client receives a damage event to a pixmap which the client has direct > rendering access to. I'm afraid I'm not sure this is going in a good direction. At the very least, the

[PATCH xserver] damage: Add screen func called before damage event delivery

2016-09-15 Thread Keith Packard
This lets the video driver flush rendering to the kernel before the client receives a damage event to a pixmap which the client has direct rendering access to. Signed-off-by: Keith Packard --- damageext/damageext.c | 1 + glamor/glamor.c | 20