From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:33519) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QgdRl-0004vO-D3 for qemu-devel@nongnu.org; Tue, 12 Jul 2011 09:56:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QgdRh-0008NF-Ft for qemu-devel@nongnu.org; Tue, 12 Jul 2011 09:56:20 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40124) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QgdRg-0008Mt-SK for qemu-devel@nongnu.org; Tue, 12 Jul 2011 09:56:17 -0400 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p6CDuFfZ028511 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Tue, 12 Jul 2011 09:56:15 -0400 From: Alon Levy Date: Tue, 12 Jul 2011 16:55:29 +0300 Message-Id: <1310478932-25370-17-git-send-email-alevy@redhat.com> In-Reply-To: <1310478932-25370-1-git-send-email-alevy@redhat.com> References: <1310478932-25370-1-git-send-email-alevy@redhat.com> Subject: [Qemu-devel] [PATCHv3] qxl-render: use update_area_async and update_area_complete List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kraxel@redhat.com So now there are two implementations chosen based on QXL_INTERFACE_MINOR: * old (spice qxl minor == 0) - use update_area, no change. * new: 1. keep an array of updated rectangles (ssd.dirty_rects) 2. update it on callback (realloc) 3. render the current one before issuing a new update_area_async --- hw/qxl-render.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++-- hw/qxl.c | 12 ++++++++++++ hw/qxl.h | 2 ++ ui/spice-display.h | 4 ++++ 4 files changed, 64 insertions(+), 2 deletions(-) diff --git a/hw/qxl-render.c b/hw/qxl-render.c index f440fe8..4862c35 100644 --- a/hw/qxl-render.c +++ b/hw/qxl-render.c @@ -103,8 +103,11 @@ static void qxl_render_update_dirty_rectangles(PCIQXLDevice *qxl, QXLRect *dirty void qxl_render_update(PCIQXLDevice *qxl) { VGACommonState *vga = &qxl->vga; - QXLRect dirty[32], update; + QXLRect update; void *ptr; +#if SPICE_INTERFACE_QXL_MINOR < 1 + QXLRect dirty[32]; +#endif if (!qxl->ssd.running) { return; @@ -112,7 +115,13 @@ void qxl_render_update(PCIQXLDevice *qxl) if (qxl->guest_primary.resized) { qxl->guest_primary.resized = 0; - + qemu_mutex_lock(&qxl->ssd.lock); + if (qxl->ssd.num_dirty_rects > 0) { + free(qxl->ssd.dirty_rects); + qxl->ssd.dirty_rects = NULL; + qxl->ssd.num_dirty_rects = 0; + } + qemu_mutex_unlock(&qxl->ssd.lock); if (qxl->guest_primary.flipped) { qemu_free(qxl->guest_primary.flipped); qxl->guest_primary.flipped = NULL; @@ -146,6 +155,19 @@ void qxl_render_update(PCIQXLDevice *qxl) dpy_resize(vga->ds); } +#if SPICE_INTERFACE_QXL_MINOR >= 1 + /* render rectangles from last update_area_async */ + qemu_mutex_lock(&qxl->ssd.lock); + if (qxl->ssd.num_dirty_rects > 0) { + qxl_render_update_dirty_rectangles(qxl, qxl->ssd.dirty_rects, + qxl->ssd.num_dirty_rects); + free(qxl->ssd.dirty_rects); + qxl->ssd.dirty_rects = NULL; + qxl->ssd.num_dirty_rects = 0; + } + qemu_mutex_unlock(&qxl->ssd.lock); +#endif + if (!qxl->guest_primary.commands) { return; } @@ -156,11 +178,33 @@ void qxl_render_update(PCIQXLDevice *qxl) update.top = 0; update.bottom = qxl->guest_primary.surface.height; +#if SPICE_INTERFACE_QXL_MINOR >= 1 + /* do a new update_area */ + qxl_spice_update_area_async(qxl, 0, &update, 1, 1); +#else memset(dirty, 0, sizeof(dirty)); qxl_spice_update_area(qxl, 0, &update, dirty, ARRAY_SIZE(dirty), 1); qxl_render_update_dirty_rectangles(qxl, dirty, ARRAY_SIZE(dirty)); +#endif +} + +#if SPICE_INTERFACE_QXL_MINOR >= 1 +void qxl_render_primary_updated(PCIQXLDevice *qxl, QXLRect *dirty, + uint32_t num_dirty) +{ + if (num_dirty == 0) { + return; + } + qemu_mutex_lock(&qxl->ssd.lock); + qxl->ssd.num_dirty_rects += num_dirty; + qxl->ssd.dirty_rects = qemu_realloc(qxl->ssd.dirty_rects, + sizeof(QXLRect) * qxl->ssd.num_dirty_rects); + memcpy(&qxl->ssd.dirty_rects[qxl->ssd.num_dirty_rects - num_dirty], + dirty, num_dirty * sizeof(QXLRect)); + qemu_mutex_unlock(&qxl->ssd.lock); } +#endif /* SPICE_INTERFACE_QXL_MINOR >= 1 */ static QEMUCursor *qxl_cursor(PCIQXLDevice *qxl, QXLCursor *cursor) { diff --git a/hw/qxl.c b/hw/qxl.c index de93efa..8a9463e 100644 --- a/hw/qxl.c +++ b/hw/qxl.c @@ -780,6 +780,17 @@ static void interface_async_complete(QXLInstance *sin, uint64_t cookie) qxl_send_events(qxl, QXL_INTERRUPT_IO_CMD); } +static void interface_update_area_complete(QXLInstance *sin, + uint32_t surface_id, struct QXLRect *updated_rects, + uint32_t num_updated_rects) +{ + PCIQXLDevice *qxl = container_of(sin, PCIQXLDevice, ssd.qxl); + + dprint(qxl, 3, "%s: %d\n", __FUNCTION__, surface_id); + if (surface_id == 0) { + qxl_render_primary_updated(qxl, updated_rects, num_updated_rects); + } +} #endif static const QXLInterface qxl_interface = { @@ -803,6 +814,7 @@ static const QXLInterface qxl_interface = { .flush_resources = interface_flush_resources, #if SPICE_INTERFACE_QXL_MINOR >= 1 .async_complete = interface_async_complete, + .update_area_complete = interface_update_area_complete, #endif }; diff --git a/hw/qxl.h b/hw/qxl.h index 2c7f94a..fad01c6 100644 --- a/hw/qxl.h +++ b/hw/qxl.h @@ -139,4 +139,6 @@ void qxl_spice_update_area_async(PCIQXLDevice *qxl, uint32_t surface_id, struct QXLRect *area, uint32_t clear_dirty_region, int is_vga); +void qxl_render_primary_updated(PCIQXLDevice *qxl, QXLRect *dirty, + uint32_t num_dirty); #endif diff --git a/ui/spice-display.h b/ui/spice-display.h index d24cca9..115aae5 100644 --- a/ui/spice-display.h +++ b/ui/spice-display.h @@ -65,6 +65,10 @@ struct SimpleSpiceDisplay { int notify; int running; +#if SPICE_INTERFACE_QXL_MINOR >= 1 + QXLRect *dirty_rects; + uint32_t num_dirty_rects; +#endif /* * All struct members below this comment can be accessed from * both spice server and qemu (iothread) context and any access -- 1.7.6