Re: [Qemu-devel] [patch 1/7] add migration_active function

2011-05-29 Thread Jes Sorensen
On 05/24/11 06:31, Marcelo Tosatti wrote:
 To query whether migration is active.
 
 Signed-off-by: Marcelo Tosatti mtosa...@redhat.com

ACK

Jes




Re: [Qemu-devel] [patch 1/7] add migration_active function

2011-05-27 Thread Kevin Wolf
Am 23.05.2011 23:31, schrieb Marcelo Tosatti:
 To query whether migration is active.
 
 Signed-off-by: Marcelo Tosatti mtosa...@redhat.com
 
 Index: qemu-block-copy/migration.c
 ===
 --- qemu-block-copy.orig/migration.c
 +++ qemu-block-copy/migration.c
 @@ -480,3 +480,13 @@ int get_migration_state(void)
  return MIG_STATE_ERROR;
  }
  }
 +
 +bool migration_active(void)
 +{
 +if (current_migration 
 +current_migration-get_status(current_migration) == 
 MIG_STATE_ACTIVE) {
 +return true;
 +}
 +
 +return false;
 +}

The very same check already exists open-coded in do_migrate(). Maybe we
should convert it now that a function is available for it?

Kevin