[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 05/15] migration: shut src return path unconditionall
From: |
Juan Quintela |
Subject: |
[Qemu-devel] [PULL 05/15] migration: shut src return path unconditionally |
Date: |
Thu, 1 Jun 2017 21:23:17 +0200 |
From: Peter Xu <address@hidden>
We were do the shutting off only for postcopy. Now we do this as long as
the source return path is there.
Moving the cleanup of from_src_file there too.
Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>
---
migration/migration.c | 8 +++++++-
migration/postcopy-ram.c | 1 -
2 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/migration/migration.c b/migration/migration.c
index b90e399..5d9ccf1 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -134,10 +134,17 @@ void migration_incoming_state_destroy(void)
struct MigrationIncomingState *mis = migration_incoming_get_current();
if (mis->to_src_file) {
+ /* Tell source that we are done */
+ migrate_send_rp_shut(mis, qemu_file_get_error(mis->from_src_file) !=
0);
qemu_fclose(mis->to_src_file);
mis->to_src_file = NULL;
}
+ if (mis->from_src_file) {
+ qemu_fclose(mis->from_src_file);
+ mis->from_src_file = NULL;
+ }
+
qemu_event_destroy(&mis->main_thread_load_event);
}
@@ -435,7 +442,6 @@ static void process_incoming_migration_co(void *opaque)
exit(EXIT_FAILURE);
}
- qemu_fclose(f);
free_xbzrle_decoded_buf();
mis->bh = qemu_bh_new(process_incoming_migration_bh, mis);
diff --git a/migration/postcopy-ram.c b/migration/postcopy-ram.c
index 3f9ae1b..5ceb623 100644
--- a/migration/postcopy-ram.c
+++ b/migration/postcopy-ram.c
@@ -333,7 +333,6 @@ int postcopy_ram_incoming_cleanup(MigrationIncomingState
*mis)
}
postcopy_state_set(POSTCOPY_INCOMING_END);
- migrate_send_rp_shut(mis, qemu_file_get_error(mis->from_src_file) != 0);
if (mis->postcopy_tmp_page) {
munmap(mis->postcopy_tmp_page, mis->largest_page_size);
--
2.9.4
- [Qemu-devel] [PULL 00/15] Migration PULL request, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 01/15] migration: Use savevm_handlers instead of loadvm copy, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 03/15] migration: Remove section_id parameter from vmstate_load, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 02/15] migration: loadvm handlers are not used, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 04/15] migration: fix leak of src file on dst, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 05/15] migration: shut src return path unconditionally,
Juan Quintela <=
- [Qemu-devel] [PULL 06/15] migration: Remove unneeded includes of migration/vmstate.h, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 08/15] migration: Export exec.c functions in its own file, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 07/15] migration: Split qemu-file.h, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 10/15] migration: Export socket.c functions in its own file, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 11/15] migration: Export tls.c functions in its own file, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 09/15] migration: Export fd.c functions in its own file, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 12/15] migration: Export rdma.c functions in its own file, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 13/15] migration: Create include for migration snapshots, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 14/15] migration: Export ram.c functions in its own file, Juan Quintela, 2017/06/01
- [Qemu-devel] [PULL 15/15] migration: Move include/migration/block.h into migration/, Juan Quintela, 2017/06/01