[PATCH v2] compositor: fix starvation of wl_buffer::release

Derek Foreman derekf at osg.samsung.com
Tue Sep 26 16:33:08 UTC 2017


On 2017-09-26 09:23 AM, Pekka Paalanen wrote:
> On Tue, 26 Sep 2017 14:26:36 +0100
> Daniel Stone <daniel at fooishbar.org> wrote:
> 
>> On 26 September 2017 at 14:09, Matt Hoosier <matt.hoosier at gmail.com> wrote:
>>> This change replaces a queued emission of buffer-release events (which
>>> is prone to starvation) with a regular event emission. This means that
>>> client programs no longer need to secretly install surface frame
>>> listeners just to guarantee that they get correctly notified of buffer
>>> lifecycle events.
>>
>> File under, 'good ideas that didn't quite work out'.
>>
>> Reviewed-by: Daniel Stone <daniels at collabora.com>
> 
> Indeed.
> 
> Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> I should remember to push this this week, unless someone else beats me
> to it. I didn't push it immediately to see if anyone else will comment.

The queued release has cause me some trouble in the past, and all the 
reasons I've seen for removing it are sound.

I've added my Rb and pushed it.

Now to do the same for Enlightenmen/EFL...

Thanks,
Derek

> 
> Thanks,
> pq
> 
> 
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel
> 



More information about the wayland-devel mailing list