Re: [PATCH] Fix re-entrancy issues in wl_display_iterate().

2012-04-14 Thread Hannu Lyytinen
On Sat, 2012-04-14 at 12:26 +0300, Pekka Paalanen wrote: > Does something prevent you from using the deferred task pattern > like upstream uses? By your explanation, it sounds like it would > achieve exactly the same (you have no other events coming when > you wait for your callback), and avoid a v

Re: [PATCH] Fix re-entrancy issues in wl_display_iterate().

2012-04-14 Thread Pekka Paalanen
On Sat, 14 Apr 2012 00:43:17 +0300 Hannu Lyytinen wrote: > On Fri, 2012-04-13 at 11:49 +0300, Pekka Paalanen wrote: > > On Fri, 13 Apr 2012 01:08:29 -0700 (PDT) > > yan.w...@linux.intel.com wrote: > > > wl_display_iterate will be called in multiply threads? > > > Normally, I think it may be calle

Re: [PATCH] Fix re-entrancy issues in wl_display_iterate().

2012-04-13 Thread Hannu Lyytinen
On Fri, 2012-04-13 at 11:49 +0300, Pekka Paalanen wrote: > On Fri, 13 Apr 2012 01:08:29 -0700 (PDT) > yan.w...@linux.intel.com wrote: > > wl_display_iterate will be called in multiply threads? Normally, I think > > it may be called in message loop thread. > > Thanks. > > It does not seem to be abo

RE: [PATCH] Fix re-entrancy issues in wl_display_iterate().

2012-04-13 Thread andy.nichols
@lists.freedesktop.org] on behalf of ext yan.w...@linux.intel.com [yan.w...@linux.intel.com] Sent: Friday, April 13, 2012 10:08 AM To: Hannu Lyytinen Cc: Hannu Lyytinen; wayland-devel@lists.freedesktop.org Subject: Re: [PATCH] Fix re-entrancy issues in wl_display_iterate(). > This fixes a possible re-entra

Re: [PATCH] Fix re-entrancy issues in wl_display_iterate().

2012-04-13 Thread Pekka Paalanen
On Fri, 13 Apr 2012 01:08:29 -0700 (PDT) yan.w...@linux.intel.com wrote: > > This fixes a possible re-entrancy issue in case of nested requests. > > > > --- > > src/connection.c |8 +++- > > src/wayland-client.c |6 -- > > src/wayland-private.h |1 + > > 3 files changed,

Re: [PATCH] Fix re-entrancy issues in wl_display_iterate().

2012-04-13 Thread yan . wang
> This fixes a possible re-entrancy issue in case of nested requests. > > --- > src/connection.c |8 +++- > src/wayland-client.c |6 -- > src/wayland-private.h |1 + > 3 files changed, 12 insertions(+), 3 deletions(-) > > diff --git a/src/connection.c b/src/connection.c >