Re: [Spice-devel] synchronous io considered harmful

2011-06-19 Thread Alon Levy
On Sun, Jun 19, 2011 at 10:43:32AM +0300, Yonit Halperin wrote: > Hi, > On 06/17/2011 07:12 PM, Alon Levy wrote: > >On Fri, Jun 17, 2011 at 09:34:12AM +0200, Gerd Hoffmann wrote: > >>On 06/16/11 21:21, Alon Levy wrote: > >>>Hi, > >>> > >>>I just had a short conversation with Marc-Andre about a bug

Re: [Spice-devel] synchronous io considered harmful

2011-06-19 Thread Yonit Halperin
Hi, On 06/17/2011 07:12 PM, Alon Levy wrote: On Fri, Jun 17, 2011 at 09:34:12AM +0200, Gerd Hoffmann wrote: On 06/16/11 21:21, Alon Levy wrote: Hi, I just had a short conversation with Marc-Andre about a bug in spice/qxl and it turns out there are a couple of possible solutions and I require s

Re: [Spice-devel] synchronous io considered harmful

2011-06-17 Thread John A. Sullivan III
On Fri, 2011-06-17 at 09:34 +0200, Gerd Hoffmann wrote: > On 06/16/11 21:21, Alon Levy wrote: > > Hi, > > > > I just had a short conversation with Marc-Andre about a bug in > > spice/qxl and it turns out there are a couple of possible solutions > > and I require some more input. > > > > The problem

Re: [Spice-devel] synchronous io considered harmful

2011-06-17 Thread Gerd Hoffmann
Hi, Yes. My current thinking (and I'm continuing with UPDATE_AREA as the example, but like you noted this needs to be done for any of the other IO's that are causing possible delays, like DESTROY_PRIMARY, CREATE_PRIMARY) is to add UPDATE_AREA_ASYNC to not break current drivers and be able to d

Re: [Spice-devel] synchronous io considered harmful

2011-06-17 Thread Alon Levy
On Fri, Jun 17, 2011 at 09:34:12AM +0200, Gerd Hoffmann wrote: > On 06/16/11 21:21, Alon Levy wrote: > >Hi, > > > >I just had a short conversation with Marc-Andre about a bug in > >spice/qxl and it turns out there are a couple of possible solutions > >and I require some more input. > > > >The probl

Re: [Spice-devel] synchronous io considered harmful

2011-06-17 Thread Gerd Hoffmann
On 06/16/11 21:21, Alon Levy wrote: Hi, I just had a short conversation with Marc-Andre about a bug in spice/qxl and it turns out there are a couple of possible solutions and I require some more input. The problem: spice was stalling in flush_display_commands caused by QXL_UPDATE_AREA. Since th

Re: [Spice-devel] synchronous io considered harmful

2011-06-16 Thread Alon Levy
On Thu, Jun 16, 2011 at 09:21:13PM +0200, Alon Levy wrote: > Hi, > > I just had a short conversation with Marc-Andre about a bug in spice/qxl and > it turns out there are a couple of possible solutions and I require some more > input. > Forgot: this is the bug: https://bugzilla.redhat.com/sho

[Spice-devel] synchronous io considered harmful

2011-06-16 Thread Alon Levy
Hi, I just had a short conversation with Marc-Andre about a bug in spice/qxl and it turns out there are a couple of possible solutions and I require some more input. The problem: spice was stalling in flush_display_commands caused by QXL_UPDATE_AREA. Since the vcpu thread is holding the glob