Commit 5089e186 authored by Dr. David Alan Gilbert's avatar Dr. David Alan Gilbert
Browse files

migration: Reset rather than destroy main_thread_load_event 



migration_incoming_state_destroy doesn't really destroy, it cleans up.
After a loadvm it's called, but the loadvm command can be run twice,
and so destroying an init-once mutex breaks on the second loadvm.

Reported-by: default avatarStafford Horne <shorne@gmail.com>
Signed-off-by: default avatarDr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20170825141940.20740-2-dgilbert@redhat.com>
Reviewed-by: default avatarPeter Xu <peterx@redhat.com>
Tested-by: default avatarStafford Horne <shorne@gmail.com>
Signed-off-by: default avatarDr. David Alan Gilbert <dgilbert@redhat.com>
parent 505d786f

migration/migration.c

+1 −1
Original line number Diff line number Diff line
@@ -166,7 +166,7 @@ void migration_incoming_state_destroy(void) 
        mis->from_src_file = NULL;

    }



    qemu_event_destroy(&mis->main_thread_load_event);

    qemu_event_reset(&mis->main_thread_load_event);

}



static void migrate_generate_event(int new_state)