Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-03-05 Thread Alon Levy
On Tue, Feb 28, 2012 at 05:05:32PM -0300, Luiz Capitulino wrote: > On Fri, 24 Feb 2012 23:22:05 +0200 > Alon Levy wrote: > > > This is an across the board change since I wanted to keep the existing > > (good imo) single graphic_console_init callback setter, instead of > > introducing a new cb tha

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-03-05 Thread Alon Levy
On Tue, Feb 28, 2012 at 05:05:32PM -0300, Luiz Capitulino wrote: > On Fri, 24 Feb 2012 23:22:05 +0200 > Alon Levy wrote: > > > This is an across the board change since I wanted to keep the existing > > (good imo) single graphic_console_init callback setter, instead of > > introducing a new cb tha

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-29 Thread Alon Levy
On Wed, Feb 29, 2012 at 11:58:37AM -0300, Luiz Capitulino wrote: > On Wed, 29 Feb 2012 10:15:53 +0200 > Alon Levy wrote: > > > On Tue, Feb 28, 2012 at 05:10:39PM -0300, Luiz Capitulino wrote: > > > On Sat, 25 Feb 2012 10:46:07 +0200 > > > Alon Levy wrote: > > > > > > > On Fri, Feb 24, 2012 at 0

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-29 Thread Luiz Capitulino
On Wed, 29 Feb 2012 10:39:29 +0200 Alon Levy wrote: > On Tue, Feb 28, 2012 at 05:05:32PM -0300, Luiz Capitulino wrote: > > On Fri, 24 Feb 2012 23:22:05 +0200 > > Alon Levy wrote: > > > > > This is an across the board change since I wanted to keep the existing > > > (good imo) single graphic_con

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-29 Thread Luiz Capitulino
On Wed, 29 Feb 2012 10:15:53 +0200 Alon Levy wrote: > On Tue, Feb 28, 2012 at 05:10:39PM -0300, Luiz Capitulino wrote: > > On Sat, 25 Feb 2012 10:46:07 +0200 > > Alon Levy wrote: > > > > > On Fri, Feb 24, 2012 at 04:40:15PM -0600, Anthony Liguori wrote: > > > > On 02/24/2012 03:22 PM, Alon Levy

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-29 Thread Luiz Capitulino
On Wed, 29 Feb 2012 08:49:10 +0100 Gerd Hoffmann wrote: > Hi, > > >> There is a hang possible with the current screendump command, qxl, a > >> spice client using libvirt and spice-gtk such as virt-viewer / > >> remote-viewer, where you have: > >> 1. libvirt waiting for screendump to complete >

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-29 Thread Alon Levy
On Tue, Feb 28, 2012 at 05:05:32PM -0300, Luiz Capitulino wrote: > On Fri, 24 Feb 2012 23:22:05 +0200 > Alon Levy wrote: > > > This is an across the board change since I wanted to keep the existing > > (good imo) single graphic_console_init callback setter, instead of > > introducing a new cb tha

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-29 Thread Alon Levy
On Tue, Feb 28, 2012 at 05:10:39PM -0300, Luiz Capitulino wrote: > On Sat, 25 Feb 2012 10:46:07 +0200 > Alon Levy wrote: > > > On Fri, Feb 24, 2012 at 04:40:15PM -0600, Anthony Liguori wrote: > > > On 02/24/2012 03:22 PM, Alon Levy wrote: > > > >This is an across the board change since I wanted t

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-28 Thread Gerd Hoffmann
Hi, >> There is a hang possible with the current screendump command, qxl, a >> spice client using libvirt and spice-gtk such as virt-viewer / >> remote-viewer, where you have: >> 1. libvirt waiting for screendump to complete >> 2. screendump waiting for spice server thread to render >> 3. spice

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-28 Thread Luiz Capitulino
On Sat, 25 Feb 2012 10:46:07 +0200 Alon Levy wrote: > On Fri, Feb 24, 2012 at 04:40:15PM -0600, Anthony Liguori wrote: > > On 02/24/2012 03:22 PM, Alon Levy wrote: > > >This is an across the board change since I wanted to keep the existing > > >(good imo) single graphic_console_init callback sett

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-28 Thread Luiz Capitulino
On Fri, 24 Feb 2012 23:22:05 +0200 Alon Levy wrote: > This is an across the board change since I wanted to keep the existing > (good imo) single graphic_console_init callback setter, instead of > introducing a new cb that isn't set by it but instead by a second > initialization function. > > Sig

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-25 Thread Alon Levy
On Fri, Feb 24, 2012 at 04:40:15PM -0600, Anthony Liguori wrote: > On 02/24/2012 03:22 PM, Alon Levy wrote: > >This is an across the board change since I wanted to keep the existing > >(good imo) single graphic_console_init callback setter, instead of > >introducing a new cb that isn't set by it bu

Re: [Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-24 Thread Anthony Liguori
On 02/24/2012 03:22 PM, Alon Levy wrote: This is an across the board change since I wanted to keep the existing (good imo) single graphic_console_init callback setter, instead of introducing a new cb that isn't set by it but instead by a second initialization function. Signed-off-by: Alon Levy

[Qemu-devel] [PATCH 4/4] add qmp screendump-async

2012-02-24 Thread Alon Levy
This is an across the board change since I wanted to keep the existing (good imo) single graphic_console_init callback setter, instead of introducing a new cb that isn't set by it but instead by a second initialization function. Signed-off-by: Alon Levy --- console.c| 25 ++