Re: [Spice-devel] [PATCH 11/11] worker: remove assertion on alloc_drawable

2015-10-30 Thread Frediano Ziglio
> > On Fri, 2015-10-30 at 13:23 -0400, Frediano Ziglio wrote: > > > > > > On Thu, Oct 29, 2015 at 04:24:27PM -0400, Frediano Ziglio wrote: > > > > > > > > > > From: Marc-André Lureau > > > > > > > > > > There is no guarantee in the code that this can't be hit, so we > > > > > should > > > > >

Re: [Spice-devel] [PATCH 11/11] worker: remove assertion on alloc_drawable

2015-10-30 Thread Jonathon Jongsma
On Fri, 2015-10-30 at 13:23 -0400, Frediano Ziglio wrote: > > > > On Thu, Oct 29, 2015 at 04:24:27PM -0400, Frediano Ziglio wrote: > > > > > > > > From: Marc-André Lureau > > > > > > > > There is no guarantee in the code that this can't be hit, so we > > > > should > > > > cope with it (the con

Re: [Spice-devel] [PATCH 11/11] worker: remove assertion on alloc_drawable

2015-10-30 Thread Frediano Ziglio
> > On Thu, Oct 29, 2015 at 04:24:27PM -0400, Frediano Ziglio wrote: > > > > > > From: Marc-André Lureau > > > > > > There is no guarantee in the code that this can't be hit, so we should > > > cope with it (the condition can be reached easily by running the server > > > without waiting for bl

Re: [Spice-devel] [PATCH 11/11] worker: remove assertion on alloc_drawable

2015-10-30 Thread Christophe Fergeau
On Thu, Oct 29, 2015 at 04:24:27PM -0400, Frediano Ziglio wrote: > > > > From: Marc-André Lureau > > > > There is no guarantee in the code that this can't be hit, so we should > > cope with it (the condition can be reached easily by running the server > > without waiting for blocked clients or p

Re: [Spice-devel] [PATCH 11/11] worker: remove assertion on alloc_drawable

2015-10-29 Thread Frediano Ziglio
> > From: Marc-André Lureau > > There is no guarantee in the code that this can't be hit, so we should > cope with it (the condition can be reached easily by running the server > without waiting for blocked clients or pipe size) > > The following commit will attempt to address this. > --- > se

[Spice-devel] [PATCH 11/11] worker: remove assertion on alloc_drawable

2015-10-29 Thread Frediano Ziglio
From: Marc-André Lureau There is no guarantee in the code that this can't be hit, so we should cope with it (the condition can be reached easily by running the server without waiting for blocked clients or pipe size) The following commit will attempt to address this. --- server/red_worker.c | 3