From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:58164) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QgzHS-0001Wo-5v for qemu-devel@nongnu.org; Wed, 13 Jul 2011 09:15:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QgzHP-0000Np-Q0 for qemu-devel@nongnu.org; Wed, 13 Jul 2011 09:15:10 -0400 Received: from mx1.redhat.com ([209.132.183.28]:47486) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QgzHP-0000Mk-Ef for qemu-devel@nongnu.org; Wed, 13 Jul 2011 09:15:07 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p6DDF6s3030683 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 13 Jul 2011 09:15:06 -0400 Date: Wed, 13 Jul 2011 10:15:04 -0300 From: Luiz Capitulino Message-ID: <20110713101504.66fc9a16@doriath> In-Reply-To: <20110713125655.GM28430@bow.tlv.redhat.com> References: <1310478932-25370-1-git-send-email-alevy@redhat.com> <1310478932-25370-14-git-send-email-alevy@redhat.com> <4E1D44DB.5090803@redhat.com> <20110713092909.GF28430@bow.tlv.redhat.com> <4E1D766C.1090308@redhat.com> <20110713112916.GL28430@bow.tlv.redhat.com> <20110713093326.272983a1@doriath> <20110713125655.GM28430@bow.tlv.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCHv3] qxl-render/qxl: split out qxl_save_ppm List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alon Levy Cc: Gerd Hoffmann , qemu-devel@nongnu.org On Wed, 13 Jul 2011 15:56:55 +0300 Alon Levy wrote: > On Wed, Jul 13, 2011 at 09:33:26AM -0300, Luiz Capitulino wrote: > > On Wed, 13 Jul 2011 14:29:16 +0300 > > Alon Levy wrote: > > > > > On Wed, Jul 13, 2011 at 12:41:48PM +0200, Gerd Hoffmann wrote: > > > > On 07/13/11 11:29, Alon Levy wrote: > > > > >On Wed, Jul 13, 2011 at 09:10:19AM +0200, Gerd Hoffmann wrote: > > > > >>On 07/12/11 15:55, Alon Levy wrote: > > > > >>>Later the save will happen asynchronously on surface_updated callback. > > > > >> > > > > >>Hmm. I can see why you are doing that. It makes the file being > > > > >>written *after* the monitor command finishes though, which I think > > > > >>we should avoid. > > > > > > > > > >I think the simplest thing would be to add a specific cond for this - ppm_save_filename_cond. ok? > > > > > > > > Not sure. Luiz, do we have async monitor commands meanwhile? > > > > > > > > Background: screendump for qxl vga can take a while as the > > > > spice-server might have to render everything first ... > > > > > > I'd rather try the MONITOR_CMD_ASYNC thing then the cond variable, it's becoming pretty > > > ugly. > > > > IIRC, that interface doesn't work as expected and is going to be replaced > > by the QAPI's one. > > In what way is in wrong? it seems to work fine here - monitor is blocked until > I call the callback, after which it returns. Didn't test the qmp part though. One problem is that the error is global and could be overwritten by other handlers. Another problem is that there's no way for a client to kill an async handler that got stuck, this could cause a client to wait for the handler forever. > > > > > > Also I guess what Daniel described is possible, but it changes the usage of screendump > > > even more. Is turning do_screen_dump to async viable? I think I'll work on it. > > > > > > > > > > > cheers, > > > > Gerd > > > > > > > > > >