Re: [PATCH] Revert "compositor: Queue buffer.release instead of sending immediately"

2013-09-11 Thread Daniel Stone
Hi, On 11 September 2013 17:15, Kristian Høgsberg wrote: > On Wed, Sep 11, 2013 at 02:54:47PM -0400, Daniel Stone wrote: >> Yeah, this is good to me. Media pipelines also really want release >> events ASAP, so they can reuse hardware-decoder buffers. > > The problem is that this will introduce a

Re: [PATCH] Revert "compositor: Queue buffer.release instead of sending immediately"

2013-09-11 Thread Kristian Høgsberg
On Wed, Sep 11, 2013 at 02:54:47PM -0400, Daniel Stone wrote: > Hi, > > On 11 September 2013 14:31, Neil Roberts wrote: > > This reverts commit eccef6aadd142103ed151883e61c0e7a2fd98639. > > > > Queuing the buffer release event instead of posting it immediately > > causes problems if the client is

Re: [PATCH] Revert "compositor: Queue buffer.release instead of sending immediately"

2013-09-11 Thread Daniel Stone
Hi, On 11 September 2013 14:31, Neil Roberts wrote: > This reverts commit eccef6aadd142103ed151883e61c0e7a2fd98639. > > Queuing the buffer release event instead of posting it immediately > causes problems if the client is not installing a frame callback and > instead is waiting for the buffer rel

[PATCH] Revert "compositor: Queue buffer.release instead of sending immediately"

2013-09-11 Thread Neil Roberts
This reverts commit eccef6aadd142103ed151883e61c0e7a2fd98639. Queuing the buffer release event instead of posting it immediately causes problems if the client is not installing a frame callback and instead is waiting for the buffer release events to throttle its rendering. This will happen in Mesa