[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 21/22] migration: Export a function that tells if th
From: |
Juan Quintela |
Subject: |
[Qemu-devel] [PATCH 21/22] migration: Export a function that tells if the migration has finished correctly |
Date: |
Wed, 23 Feb 2011 01:44:35 +0100 |
This will allows us to hide the status values.
Signed-off-by: Juan Quintela <address@hidden>
---
migration.c | 4 ++--
migration.h | 2 +-
ui/spice-core.c | 4 +---
3 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/migration.c b/migration.c
index 312a029..383ebaf 100644
--- a/migration.c
+++ b/migration.c
@@ -335,9 +335,9 @@ void remove_migration_state_change_notifier(Notifier
*notify)
notifier_list_remove(&migration_state_notifiers, notify);
}
-int get_migration_state(void)
+bool migration_has_finished(void)
{
- return current_migration.state;
+ return current_migration.state == MIG_STATE_COMPLETED;
}
void migrate_fd_connect(MigrationState *s)
diff --git a/migration.h b/migration.h
index 6477b51..9457807 100644
--- a/migration.h
+++ b/migration.h
@@ -82,6 +82,6 @@ void migrate_fd_connect(MigrationState *s);
void add_migration_state_change_notifier(Notifier *notify);
void remove_migration_state_change_notifier(Notifier *notify);
-int get_migration_state(void);
+bool migration_has_finished(void);
#endif
diff --git a/ui/spice-core.c b/ui/spice-core.c
index 1aa1a5e..997603d 100644
--- a/ui/spice-core.c
+++ b/ui/spice-core.c
@@ -422,9 +422,7 @@ void do_info_spice(Monitor *mon, QObject **ret_data)
static void migration_state_notifier(Notifier *notifier)
{
- int state = get_migration_state();
-
- if (state == MIG_STATE_COMPLETED) {
+ if (migration_has_finished()) {
#if SPICE_SERVER_VERSION >= 0x000701 /* 0.7.1 */
spice_server_migrate_switch(spice_server);
#endif
--
1.7.4
- Re: [Qemu-devel] [PATCH 14/22] migration: Remove get_status() accessor, (continued)
- [Qemu-devel] [PATCH 16/22] migration: Move exported functions to the end of the file, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 17/22] migration: use global variable directly, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 18/22] migration: another case of global variable assigned to local one, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 19/22] migration: convert current_migration from pointer to struct, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 21/22] migration: Export a function that tells if the migration has finished correctly,
Juan Quintela <=
- [Qemu-devel] [PATCH 22/22] migration: Make state definitions local, Juan Quintela, 2011/02/22
- Re: [Qemu-devel] [PATCH 22/22] migration: Make state definitions local, Yoshiaki Tamura, 2011/02/23
- [Qemu-devel] Re: [PATCH 22/22] migration: Make state definitions local, Juan Quintela, 2011/02/23
- Re: [Qemu-devel] Re: [PATCH 22/22] migration: Make state definitions local, Yoshiaki Tamura, 2011/02/23
- [Qemu-devel] Re: [PATCH 22/22] migration: Make state definitions local, Juan Quintela, 2011/02/24
- Re: [Qemu-devel] Re: [PATCH 22/22] migration: Make state definitions local, Anthony Liguori, 2011/02/24
- Re: [Qemu-devel] Re: [PATCH 22/22] migration: Make state definitions local, Yoshiaki Tamura, 2011/02/24
- Re: [Qemu-devel] Re: [PATCH 22/22] migration: Make state definitions local, Yoshiaki Tamura, 2011/02/24
[Qemu-devel] [PATCH 20/22] migration: Use bandwidth_limit directly, Juan Quintela, 2011/02/22