Re: [Qemu-devel] [PATCH v2] qxl: allow QXL_IO_LOG also in vga

2011-06-29 Thread Alon Levy
On Wed, Jun 29, 2011 at 12:02:30PM +0200, Gerd Hoffmann wrote: > Hi, > > >Yeah, my bad. I did something wierd, and missed this. I'll send a v3. Do you > >have other comments? > > I stopped looking after figuring stuff doesn't apply to master. > > >Unrelated, but have you looked at the async p

Re: [Qemu-devel] [PATCH v2] qxl: allow QXL_IO_LOG also in vga

2011-06-29 Thread Gerd Hoffmann
Hi, Yeah, my bad. I did something wierd, and missed this. I'll send a v3. Do you have other comments? I stopped looking after figuring stuff doesn't apply to master. Unrelated, but have you looked at the async patches that I sent, and yonit's comments? - (me) catch EINTR and EAGAIN in

Re: [Qemu-devel] [PATCH v2] qxl: allow QXL_IO_LOG also in vga

2011-06-29 Thread Alon Levy
On Wed, Jun 29, 2011 at 11:22:05AM +0200, Gerd Hoffmann wrote: > Hi, > > >The driver may change us to vga mode and still issue a QXL_IO_LOG, > >which we can easily support. > > >@@ -1001,9 +1001,9 @@ static void ioport_write(void *opaque, uint32_t addr, > >uint32_t val) > >-case QXL_IO_LOG

Re: [Qemu-devel] [PATCH v2] qxl: allow QXL_IO_LOG also in vga

2011-06-29 Thread Gerd Hoffmann
Hi, The driver may change us to vga mode and still issue a QXL_IO_LOG, which we can easily support. @@ -1001,9 +1001,9 @@ static void ioport_write(void *opaque, uint32_t addr, uint32_t val) -case QXL_IO_LOG: case QXL_IO_CREATE_PRIMARY_ASYNC: case QXL_IO_UPDATE_IRQ: +cas