Re: [Qemu-devel] [PATCH 39/41] migration: move contents of migration_close to migrate_fd_cleanup

[Orit Wasserman]

On 02/15/2013 07:47 PM, Paolo Bonzini wrote:
> With this patch, the migration_file is not needed anymore.
> 
> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
> ---
>  migration.c |   17 +++++++----------
>  1 files changed, 7 insertions(+), 10 deletions(-)
> 
> diff --git a/migration.c b/migration.c
> index 7c1671f..c9a5978 100644
> --- a/migration.c
> +++ b/migration.c
> @@ -271,6 +271,12 @@ static void migrate_fd_cleanup(void *opaque)
>          DPRINTF("closing file\n");
>          qemu_fclose(s->file);
>          s->file = NULL;
> +
> +        qemu_mutex_unlock_iothread();
> +        qemu_thread_join(&s->thread);
> +        qemu_mutex_lock_iothread();
> +
> +        migrate_fd_close(s);
>      }
>  
>      assert(s->migration_file == NULL);
> @@ -506,16 +512,7 @@ static int migration_put_buffer(void *opaque, const 
> uint8_t *buf,
>  
>  static int migration_close(void *opaque)
>  {
> -    MigrationState *s = opaque;
> -
> -    DPRINTF("closing\n");
> -
> -    qemu_mutex_unlock_iothread();
> -    qemu_thread_join(&s->thread);
> -    qemu_mutex_lock_iothread();
> -    assert (s->state != MIG_STATE_ACTIVE);
> -
> -    return migrate_fd_close(s);
> +    return 0;
>  }
>  
>  static int migration_get_fd(void *opaque)
> 
Reviewed-by: Orit Wasserman <owass...@redhat.com>