Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-26 Thread Alon Levy
On Tue, Oct 25, 2011 at 04:41:18PM -0200, Luiz Capitulino wrote: On Tue, 25 Oct 2011 15:21:11 +0200 Alon Levy al...@redhat.com wrote: On Tue, Oct 25, 2011 at 10:51:30AM -0200, Luiz Capitulino wrote: On Tue, 25 Oct 2011 12:13:09 +0200 Alon Levy al...@redhat.com wrote: On Mon,

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-25 Thread Gerd Hoffmann
Hi, If all you need here is to delay sending the response, then maybe the current interface could work (although I honestly don't trust it and regret not having dropped it). Otherwise our only choice would be to work on getting the QAPI async support merged. A delayed monitor response is

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-25 Thread Alon Levy
On Mon, Oct 24, 2011 at 10:31:48PM -0200, Luiz Capitulino wrote: On Mon, 24 Oct 2011 19:29:37 +0200 Alon Levy al...@redhat.com wrote: On Mon, Oct 24, 2011 at 01:45:16PM -0200, Luiz Capitulino wrote: On Mon, 24 Oct 2011 17:13:14 +0200 Gerd Hoffmann kra...@redhat.com wrote: On

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-25 Thread Luiz Capitulino
On Tue, 25 Oct 2011 12:13:09 +0200 Alon Levy al...@redhat.com wrote: On Mon, Oct 24, 2011 at 10:31:48PM -0200, Luiz Capitulino wrote: On Mon, 24 Oct 2011 19:29:37 +0200 Alon Levy al...@redhat.com wrote: On Mon, Oct 24, 2011 at 01:45:16PM -0200, Luiz Capitulino wrote: On Mon, 24 Oct

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-25 Thread Alon Levy
On Tue, Oct 25, 2011 at 10:51:30AM -0200, Luiz Capitulino wrote: On Tue, 25 Oct 2011 12:13:09 +0200 Alon Levy al...@redhat.com wrote: On Mon, Oct 24, 2011 at 10:31:48PM -0200, Luiz Capitulino wrote: On Mon, 24 Oct 2011 19:29:37 +0200 Alon Levy al...@redhat.com wrote: On Mon,

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-25 Thread Luiz Capitulino
On Tue, 25 Oct 2011 15:21:11 +0200 Alon Levy al...@redhat.com wrote: On Tue, Oct 25, 2011 at 10:51:30AM -0200, Luiz Capitulino wrote: On Tue, 25 Oct 2011 12:13:09 +0200 Alon Levy al...@redhat.com wrote: On Mon, Oct 24, 2011 at 10:31:48PM -0200, Luiz Capitulino wrote: On Mon, 24 Oct

[Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-24 Thread Alon Levy
Make screen_dump monitor command an async command to allow next for qxl to implement it as a initiating call to red_worker and completion on callback, to fix a deadlock when issueing a screendump command via libvirt while connected with a libvirt controlled spice-gtk client. This patch introduces

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-24 Thread Gerd Hoffmann
On 10/24/11 14:02, Alon Levy wrote: Make screen_dump monitor command an async command to allow next for qxl to implement it as a initiating call to red_worker and completion on callback, to fix a deadlock when issueing a screendump command via libvirt while connected with a libvirt controlled

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-24 Thread Luiz Capitulino
On Mon, 24 Oct 2011 17:13:14 +0200 Gerd Hoffmann kra...@redhat.com wrote: On 10/24/11 14:02, Alon Levy wrote: Make screen_dump monitor command an async command to allow next for qxl to implement it as a initiating call to red_worker and completion on callback, to fix a deadlock when

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-24 Thread Alon Levy
On Mon, Oct 24, 2011 at 01:45:16PM -0200, Luiz Capitulino wrote: On Mon, 24 Oct 2011 17:13:14 +0200 Gerd Hoffmann kra...@redhat.com wrote: On 10/24/11 14:02, Alon Levy wrote: Make screen_dump monitor command an async command to allow next for qxl to implement it as a initiating call to

Re: [Qemu-devel] [PATCH 1/5] monitor: screen_dump async

2011-10-24 Thread Luiz Capitulino
On Mon, 24 Oct 2011 19:29:37 +0200 Alon Levy al...@redhat.com wrote: On Mon, Oct 24, 2011 at 01:45:16PM -0200, Luiz Capitulino wrote: On Mon, 24 Oct 2011 17:13:14 +0200 Gerd Hoffmann kra...@redhat.com wrote: On 10/24/11 14:02, Alon Levy wrote: Make screen_dump monitor command an