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 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 wrote: > > > > > > > On Mon, Oct 24, 2011 at 1

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 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 wrote: > > > > > On Mon, Oct 24, 2011 at 10:31:48PM -0200, Luiz Capitulino wrote: > > > > On Mon, 24 Oct 2011 19:29:37 +020

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 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 wrote: > > > > > > > On Mon, Oct 24, 2011 at 0

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 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 wrote: > > > > > On Mon, Oct 24, 2011 at 01:45:16PM -0200, Luiz Capitulino wrote: > > > > On Mon, 24 Oct 2011 17:13:14 +020

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 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 wrote: > > > > > > > On 10/24/11 14:02, Al

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 i

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 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 wrote: > > > > > On 10/24/11 14:02, Alon Levy wrote: > > > > Make screen_dump monitor command an async command to allow

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 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_wo

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 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 issueing a screen

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 controll

[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