Re: [PATCH wayland] client: ensure thread safety for wl_display_roundtrip_queue()

2016-03-03 Thread Pekka Paalanen
On Wed, 2 Mar 2016 11:01:12 -0800 Bill Spitzak wrote: > On Wed, Mar 2, 2016 at 4:25 AM, Pekka Paalanen wrote: > > > > > > bar_obj = wl_bar_new(); > > > wl_proxy_set_queue((wl_proxy *)bar_obj, queue); > > > wl_bar_add_listener(bar_obj, ...); > > > /*

Re: [PATCH wayland] client: ensure thread safety for wl_display_roundtrip_queue()

2016-03-02 Thread Pekka Paalanen
On Tue, 1 Mar 2016 11:41:21 -0800 Bill Spitzak wrote: > On Thu, Feb 18, 2016 at 4:12 AM, Pekka Paalanen wrote: > > > On Tue, 16 Feb 2016 14:26:53 -0800 > > Bill Spitzak wrote: > > > > > On Mon, Feb 15, 2016 at 7:48 PM, Jonas Ådahl

Re: [PATCH wayland] client: ensure thread safety for wl_display_roundtrip_queue()

2016-02-18 Thread Pekka Paalanen
On Tue, 16 Feb 2016 14:26:53 -0800 Bill Spitzak wrote: > On Mon, Feb 15, 2016 at 7:48 PM, Jonas Ådahl wrote: > > The proxy wrapper approach can be tested out by appling this patch: > >

Re: [PATCH wayland] client: ensure thread safety for wl_display_roundtrip_queue()

2016-02-18 Thread Bill Spitzak
On Mon, Feb 15, 2016 at 7:48 PM, Jonas Ådahl wrote: The proxy wrapper approach can be tested out by appling this patch: > https://lists.freedesktop.org/archives/wayland-devel/2015-June/023054.html Paraphrased here: > In short, instead of > > bar =

Re: [PATCH wayland] client: ensure thread safety for wl_display_roundtrip_queue()

2016-02-16 Thread Jonas Ådahl
mre (ADITG/SW1) > > Cc: wayland-devel@lists.freedesktop.org > > Subject: Re: [PATCH wayland] client: ensure thread safety for > > wl_display_roundtrip_queue() > > > > On Mon, Feb 15, 2016 at 02:19:43PM +, Ucan, Emre (ADITG/SW1) wrote: > > > We have to

RE: [PATCH wayland] client: ensure thread safety for wl_display_roundtrip_queue()

2016-02-16 Thread Ucan, Emre (ADITG/SW1)
6937 > -Original Message- > From: wayland-devel [mailto:wayland-devel- > boun...@lists.freedesktop.org] On Behalf Of Jonas Ådahl > Sent: Dienstag, 16. Februar 2016 04:49 > To: Ucan, Emre (ADITG/SW1) > Cc: wayland-devel@lists.freedesktop.org > Subject: Re: [PATCH wa

Re: [PATCH wayland] client: ensure thread safety for wl_display_roundtrip_queue()

2016-02-15 Thread Jonas Ådahl
On Mon, Feb 15, 2016 at 02:19:43PM +, Ucan, Emre (ADITG/SW1) wrote: > We have to block other threads from reading events. Otherwise other threads > may assign done event to the default queue, if they call > wl_display_read_events before setting the queue for the callback proxy. > This would

[PATCH wayland] client: ensure thread safety for wl_display_roundtrip_queue()

2016-02-15 Thread Ucan, Emre (ADITG/SW1)
We have to block other threads from reading events. Otherwise other threads may assign done event to the default queue, if they call wl_display_read_events before setting the queue for the callback proxy. This would cause a deadlock in wl_display_dispatch_queue, because the default queue is not