From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:53093) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rzy21-0000N3-9K for qemu-devel@nongnu.org; Tue, 21 Feb 2012 17:17:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Rzy1z-0005Hf-Sq for qemu-devel@nongnu.org; Tue, 21 Feb 2012 17:17:57 -0500 Received: from mx1.redhat.com ([209.132.183.28]:53600) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rzy1z-0005HT-MK for qemu-devel@nongnu.org; Tue, 21 Feb 2012 17:17:55 -0500 Date: Wed, 22 Feb 2012 00:17:46 +0200 From: Alon Levy Message-ID: <20120221221746.GZ6476@garlic> References: <1329860377-6284-1-git-send-email-alevy@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1329860377-6284-1-git-send-email-alevy@redhat.com> Subject: Re: [Qemu-devel] [Spice-devel] [RFC v4 0/9] qxl: fix hangs caused by qxl_render_update List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, kraxel@redhat.com, spice-devel@freedesktop.org, yhalperi@redhat.com, elmarco@redhat.com On Tue, Feb 21, 2012 at 11:39:28PM +0200, Alon Levy wrote: > This is the second attempt to fix this issue, as a lesson from the last time it doesn't try to use an async monitor command. So with this patchset, in qxl mode, a screendump monitor command will complete before the file is written to disk. This is much better then a hang. To fix it does require asynchronous monitor command completion. > > Changes from previous RFC (v4 to match my internal patches, it's actually the second): Forgot: depend on spice-server >= 0.8.2 Last patch has an ugly hack in the ppm_save bh, that does flipping, since I missed something somewhere.. Also the whole flipping assumes that qxl always has negative stride and the displaysurface always has positive stride, no checks anywhere.. > - dropped the DisplayAllocator > - dropping flip (Gerd Hoffman suggestion) > - dropping needless call to console_select (from Gerd) > - use a bh for the bug fix itself, previously called dpy_update > and wrote to vga->ds.surface->data from spice server thread, a no no. > - last patch uses a bh for the ppm_save as well, same problem as one > in the previous round. I'm going to work on a solution to that as discussed > on the list (either a QAPI equivalent of the old async monitor or a new really > async command with completion event that libvirt/autotest/etc would have to > use), but didn't want to wait for that with the already prolonged wait from the last > RFC. > > Please review, > > Alon Levy (8): > sdl: remove NULL check, g_malloc0 can't fail > qxl: drop qxl_spice_update_area_async definition > qxl: screen_dump in vga: do a single ppm_save > qxl: require spice >= 0.8.2 > qxl: remove flipped > qxl: introduce QXLCookie > qxl: make qxl_render_update async > qxl-render: call ppm_save on bh > > Gerd Hoffman (1): > console: don't call console_select unnecessarily > > configure | 2 +- > console.c | 6 +- > hw/qxl-render.c | 228 ++++++++++++++++++++++++++++++++++++++-------------- > hw/qxl.c | 189 ++++++++++++++++++++++++++++++------------- > hw/qxl.h | 28 ++++--- > ui/sdl.c | 4 - > ui/spice-display.c | 34 +++++---- > ui/spice-display.h | 22 +++++ > 8 files changed, 359 insertions(+), 154 deletions(-) > > -- > 1.7.9.1 > > _______________________________________________ > Spice-devel mailing list > Spice-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/spice-devel