[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [RFC PATCH 5/5] spice: turn spice "migration end" handler t
From: |
Yonit Halperin |
Subject: |
[Qemu-devel] [RFC PATCH 5/5] spice: turn spice "migration end" handler to be async |
Date: |
Tue, 5 Jun 2012 08:49:46 +0300 |
Instead of immediatly calling the notifier completion callback,
the notification handler assigns a completion callback
to spice's migration interface. Spice should call this
callback when it completes handling the migration state change.
Signed-off-by: Yonit Halperin <address@hidden>
---
ui/spice-core.c | 29 ++++++++++++++++++++++-------
1 files changed, 22 insertions(+), 7 deletions(-)
diff --git a/ui/spice-core.c b/ui/spice-core.c
index d85c212..053f06f 100644
--- a/ui/spice-core.c
+++ b/ui/spice-core.c
@@ -282,9 +282,14 @@ typedef struct SpiceMigration {
MonitorCompletion *cb;
void *opaque;
} connect_complete;
+ struct {
+ NotifiedCompletionFunc *cb;
+ void *opaque;
+ } end_complete;
} SpiceMigration;
static void migrate_connect_complete_cb(SpiceMigrateInstance *sin);
+static void migrate_end_complete_cb(SpiceMigrateInstance *sin);
static const SpiceMigrateInterface migrate_interface = {
.base.type = SPICE_INTERFACE_MIGRATION,
@@ -292,7 +297,7 @@ static const SpiceMigrateInterface migrate_interface = {
.base.major_version = SPICE_INTERFACE_MIGRATION_MAJOR,
.base.minor_version = SPICE_INTERFACE_MIGRATION_MINOR,
.migrate_connect_complete = migrate_connect_complete_cb,
- .migrate_end_complete = NULL,
+ .migrate_end_complete = migrate_end_complete_cb,
};
static SpiceMigration spice_migrate;
@@ -305,6 +310,15 @@ static void
migrate_connect_complete_cb(SpiceMigrateInstance *sin)
}
sm->connect_complete.cb = NULL;
}
+
+static void migrate_end_complete_cb(SpiceMigrateInstance *sin)
+{
+ SpiceMigration *sm = container_of(sin, SpiceMigration, sin);
+ if (sm->end_complete.cb) {
+ sm->end_complete.cb(&migrate_end_notifier, sm->end_complete.opaque);
+ }
+ sm->end_complete.cb = NULL;
+}
#endif
/* config string parsing */
@@ -492,16 +506,17 @@ static void migrate_end_notify_func(AsyncNotifier
*notifier, void *data,
void *cb_data)
{
bool success_end = *(bool *)data;
+
+#ifdef SPICE_INTERFACE_MIGRATION
+ spice_migrate.end_complete.cb = complete_cb;
+ spice_migrate.end_complete.opaque = cb_data;
+ spice_server_migrate_end(spice_server, success_end);
+#else
if (success_end) {
-#ifndef SPICE_INTERFACE_MIGRATION
spice_server_migrate_switch(spice_server);
-#else
- spice_server_migrate_end(spice_server, true);
- } else {
- spice_server_migrate_end(spice_server, false);
-#endif
}
complete_cb(notifier, cb_data);
+#endif
}
int qemu_spice_migrate_info(const char *hostname, int port, int tls_port,
--
1.7.7.6
- [Qemu-devel] [RFC PATCH 0/5] asynchronous migration state change handlers, Yonit Halperin, 2012/06/05
- [Qemu-devel] [RFC PATCH 1/5] notifiers: add support for async notifiers handlers, Yonit Halperin, 2012/06/05
- [Qemu-devel] [RFC PATCH 2/5] migration: moving migration start code to a separated routine, Yonit Halperin, 2012/06/05
- [Qemu-devel] [RFC PATCH 3/5] migration: moving migration completion code to a separated routine, Yonit Halperin, 2012/06/05
- [Qemu-devel] [RFC PATCH 5/5] spice: turn spice "migration end" handler to be async,
Yonit Halperin <=
- [Qemu-devel] [RFC PATCH 4/5] migration: replace migration state change notifier with async notifiers, Yonit Halperin, 2012/06/05
- Re: [Qemu-devel] [RFC PATCH 0/5] asynchronous migration state change handlers, Anthony Liguori, 2012/06/05
- Re: [Qemu-devel] [RFC PATCH 0/5] asynchronous migration state change handlers, Gerd Hoffmann, 2012/06/05
- Re: [Qemu-devel] [RFC PATCH 0/5] asynchronous migration state change handlers, Yonit Halperin, 2012/06/06
- Re: [Qemu-devel] [RFC PATCH 0/5] asynchronous migration state change handlers, Anthony Liguori, 2012/06/06
- Re: [Qemu-devel] [RFC PATCH 0/5] asynchronous migration state change handlers, Alon Levy, 2012/06/06
- Re: [Qemu-devel] [RFC PATCH 0/5] asynchronous migration state change handlers, Anthony Liguori, 2012/06/06
- Re: [Qemu-devel] [RFC PATCH 0/5] asynchronous migration state change handlers, Alon Levy, 2012/06/06