Blame 0215-spice-switch-to-queue-for-vga-mode-updates.patch

Hans de Goede 93b7e3
From 20b7c4b95c7f0c93000a68779e9cb9e63cff68da Mon Sep 17 00:00:00 2001
Hans de Goede 93b7e3
From: Gerd Hoffmann <kraxel@redhat.com>
Hans de Goede 93b7e3
Date: Wed, 5 Sep 2012 08:25:08 +0200
Hans de Goede 93b7e3
Subject: [PATCH 215/293] spice: switch to queue for vga mode updates
Hans de Goede 93b7e3
Hans de Goede 93b7e3
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Hans de Goede 93b7e3
---
Hans de Goede 93b7e3
 hw/qxl.c           |  6 +++---
Hans de Goede 93b7e3
 ui/spice-display.c | 25 ++++++++++++++-----------
Hans de Goede 93b7e3
 ui/spice-display.h |  3 ++-
Hans de Goede 93b7e3
 3 files changed, 19 insertions(+), 15 deletions(-)
Hans de Goede 93b7e3
Hans de Goede 93b7e3
diff --git a/hw/qxl.c b/hw/qxl.c
Hans de Goede 93b7e3
index b726c19..833cd77 100644
Hans de Goede 93b7e3
--- a/hw/qxl.c
Hans de Goede 93b7e3
+++ b/hw/qxl.c
Hans de Goede 93b7e3
@@ -597,9 +597,9 @@ static int interface_get_command(QXLInstance *sin, struct QXLCommandExt *ext)
Hans de Goede 93b7e3
     case QXL_MODE_VGA:
Hans de Goede 93b7e3
         ret = false;
Hans de Goede 93b7e3
         qemu_mutex_lock(&qxl->ssd.lock);
Hans de Goede 93b7e3
-        if (qxl->ssd.update != NULL) {
Hans de Goede 93b7e3
-            update = qxl->ssd.update;
Hans de Goede 93b7e3
-            qxl->ssd.update = NULL;
Hans de Goede 93b7e3
+        update = QTAILQ_FIRST(&qxl->ssd.updates);
Hans de Goede 93b7e3
+        if (update != NULL) {
Hans de Goede 93b7e3
+            QTAILQ_REMOVE(&qxl->ssd.updates, update, next);
Hans de Goede 93b7e3
             *ext = update->ext;
Hans de Goede 93b7e3
             ret = true;
Hans de Goede 93b7e3
         }
Hans de Goede 93b7e3
diff --git a/ui/spice-display.c b/ui/spice-display.c
Hans de Goede 93b7e3
index 99bc665..59c5fd7 100644
Hans de Goede 93b7e3
--- a/ui/spice-display.c
Hans de Goede 93b7e3
+++ b/ui/spice-display.c
Hans de Goede 93b7e3
@@ -164,7 +164,7 @@ int qemu_spice_display_is_running(SimpleSpiceDisplay *ssd)
Hans de Goede 93b7e3
 #endif
Hans de Goede 93b7e3
 }
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
-static SimpleSpiceUpdate *qemu_spice_create_update(SimpleSpiceDisplay *ssd)
Hans de Goede 93b7e3
+static void qemu_spice_create_update(SimpleSpiceDisplay *ssd)
Hans de Goede 93b7e3
 {
Hans de Goede 93b7e3
     SimpleSpiceUpdate *update;
Hans de Goede 93b7e3
     QXLDrawable *drawable;
Hans de Goede 93b7e3
@@ -175,7 +175,7 @@ static SimpleSpiceUpdate *qemu_spice_create_update(SimpleSpiceDisplay *ssd)
Hans de Goede 93b7e3
     struct timespec time_space;
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
     if (qemu_spice_rect_is_empty(&ssd->dirty)) {
Hans de Goede 93b7e3
-        return NULL;
Hans de Goede 93b7e3
+        return;
Hans de Goede 93b7e3
     };
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
     trace_qemu_spice_create_update(
Hans de Goede 93b7e3
@@ -239,7 +239,7 @@ static SimpleSpiceUpdate *qemu_spice_create_update(SimpleSpiceDisplay *ssd)
Hans de Goede 93b7e3
     cmd->data = (uintptr_t)drawable;
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
     memset(&ssd->dirty, 0, sizeof(ssd->dirty));
Hans de Goede 93b7e3
-    return update;
Hans de Goede 93b7e3
+    QTAILQ_INSERT_TAIL(&ssd->updates, update, next);
Hans de Goede 93b7e3
 }
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
 /*
Hans de Goede 93b7e3
@@ -315,6 +315,7 @@ void qemu_spice_display_init_common(SimpleSpiceDisplay *ssd, DisplayState *ds)
Hans de Goede 93b7e3
 {
Hans de Goede 93b7e3
     ssd->ds = ds;
Hans de Goede 93b7e3
     qemu_mutex_init(&ssd->lock);
Hans de Goede 93b7e3
+    QTAILQ_INIT(&ssd->updates);
Hans de Goede 93b7e3
     ssd->mouse_x = -1;
Hans de Goede 93b7e3
     ssd->mouse_y = -1;
Hans de Goede 93b7e3
     if (ssd->num_surfaces == 0) {
Hans de Goede 93b7e3
@@ -345,6 +346,8 @@ void qemu_spice_display_update(SimpleSpiceDisplay *ssd,
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
 void qemu_spice_display_resize(SimpleSpiceDisplay *ssd)
Hans de Goede 93b7e3
 {
Hans de Goede 93b7e3
+    SimpleSpiceUpdate *update;
Hans de Goede 93b7e3
+
Hans de Goede 93b7e3
     dprint(1, "%s:\n", __FUNCTION__);
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
     memset(&ssd->dirty, 0, sizeof(ssd->dirty));
Hans de Goede 93b7e3
@@ -352,9 +355,9 @@ void qemu_spice_display_resize(SimpleSpiceDisplay *ssd)
Hans de Goede 93b7e3
     ssd->conv = NULL;
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
     qemu_mutex_lock(&ssd->lock);
Hans de Goede 93b7e3
-    if (ssd->update != NULL) {
Hans de Goede 93b7e3
-        qemu_spice_destroy_update(ssd, ssd->update);
Hans de Goede 93b7e3
-        ssd->update = NULL;
Hans de Goede 93b7e3
+    while ((update = QTAILQ_FIRST(&ssd->updates)) != NULL) {
Hans de Goede 93b7e3
+        QTAILQ_REMOVE(&ssd->updates, update, next);
Hans de Goede 93b7e3
+        qemu_spice_destroy_update(ssd, update);
Hans de Goede 93b7e3
     }
Hans de Goede 93b7e3
     qemu_mutex_unlock(&ssd->lock);
Hans de Goede 93b7e3
     qemu_spice_destroy_host_primary(ssd);
Hans de Goede 93b7e3
@@ -384,8 +387,8 @@ void qemu_spice_display_refresh(SimpleSpiceDisplay *ssd)
Hans de Goede 93b7e3
     vga_hw_update();
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
     qemu_mutex_lock(&ssd->lock);
Hans de Goede 93b7e3
-    if (ssd->update == NULL) {
Hans de Goede 93b7e3
-        ssd->update = qemu_spice_create_update(ssd);
Hans de Goede 93b7e3
+    if (QTAILQ_EMPTY(&ssd->updates)) {
Hans de Goede 93b7e3
+        qemu_spice_create_update(ssd);
Hans de Goede 93b7e3
         ssd->notify++;
Hans de Goede 93b7e3
     }
Hans de Goede 93b7e3
     qemu_spice_cursor_refresh_unlocked(ssd);
Hans de Goede 93b7e3
@@ -442,9 +445,9 @@ static int interface_get_command(QXLInstance *sin, struct QXLCommandExt *ext)
Hans de Goede 93b7e3
     dprint(3, "%s:\n", __FUNCTION__);
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
     qemu_mutex_lock(&ssd->lock);
Hans de Goede 93b7e3
-    if (ssd->update != NULL) {
Hans de Goede 93b7e3
-        update = ssd->update;
Hans de Goede 93b7e3
-        ssd->update = NULL;
Hans de Goede 93b7e3
+    update = QTAILQ_FIRST(&ssd->updates);
Hans de Goede 93b7e3
+    if (update != NULL) {
Hans de Goede 93b7e3
+        QTAILQ_REMOVE(&ssd->updates, update, next);
Hans de Goede 93b7e3
         *ext = update->ext;
Hans de Goede 93b7e3
         ret = true;
Hans de Goede 93b7e3
     }
Hans de Goede 93b7e3
diff --git a/ui/spice-display.h b/ui/spice-display.h
Hans de Goede 93b7e3
index 512ab78..3fcb6fe 100644
Hans de Goede 93b7e3
--- a/ui/spice-display.h
Hans de Goede 93b7e3
+++ b/ui/spice-display.h
Hans de Goede 93b7e3
@@ -92,7 +92,7 @@ struct SimpleSpiceDisplay {
Hans de Goede 93b7e3
      * to them must be protected by the lock.
Hans de Goede 93b7e3
      */
Hans de Goede 93b7e3
     QemuMutex lock;
Hans de Goede 93b7e3
-    SimpleSpiceUpdate *update;
Hans de Goede 93b7e3
+    QTAILQ_HEAD(, SimpleSpiceUpdate) updates;
Hans de Goede 93b7e3
     QEMUCursor *cursor;
Hans de Goede 93b7e3
     int mouse_x, mouse_y;
Hans de Goede 93b7e3
 };
Hans de Goede 93b7e3
@@ -102,6 +102,7 @@ struct SimpleSpiceUpdate {
Hans de Goede 93b7e3
     QXLImage image;
Hans de Goede 93b7e3
     QXLCommandExt ext;
Hans de Goede 93b7e3
     uint8_t *bitmap;
Hans de Goede 93b7e3
+    QTAILQ_ENTRY(SimpleSpiceUpdate) next;
Hans de Goede 93b7e3
 };
Hans de Goede 93b7e3
 
Hans de Goede 93b7e3
 int qemu_spice_rect_is_empty(const QXLRect* r);
Hans de Goede 93b7e3
-- 
Hans de Goede 93b7e3
1.7.12
Hans de Goede 93b7e3