@@ -92,6 +92,9 @@ enum mig_rp_message_type {
MIG_RP_MSG_MAX
};
+/* Migration channel types */
+enum { CH_DEFAULT, CH_MULTIFD, CH_POSTCOPY };
+
/* When we add fault tolerance, we could have several
migrations at once. For now we don't need to add
dynamic creation of migration */
@@ -921,26 +924,32 @@ void migration_fd_process_incoming(QEMUFile *f)
/*
* Returns true when we want to start a new incoming migration process,
* false otherwise.
+ *
+ * All the required channels must be in place before a new incoming
+ * migration process starts.
+ * - Multifd enabled:
+ * The main channel and the multifd channels are required.
+ * - Multifd/Postcopy disabled:
+ * The main channel is required.
+ * - Postcopy enabled:
+ * We don't want to start a new incoming migration when
+ * the postcopy channel is created. Because it is created
+ * towards the end of the precopy migration.
*/
-static bool migration_should_start_incoming(bool main_channel)
+static bool migration_should_start_incoming(uint8_t channel)
{
- /* Multifd doesn't start unless all channels are established */
- if (migrate_multifd()) {
- return migration_has_all_channels();
- }
+ bool ret = false;
+
+ if (channel != CH_POSTCOPY) {
+ MigrationIncomingState *mis = migration_incoming_get_current();
+ ret = mis->from_src_file ? true : false;
- /* Preempt channel only starts when the main channel is created */
- if (migrate_postcopy_preempt()) {
- return main_channel;
+ if (ret && migrate_multifd()) {
+ ret = multifd_recv_all_channels_created();
+ }
}
- /*
- * For all the rest types of migration, we should only reach here when
- * it's the main channel that's being created, and we should always
- * proceed with this channel.
- */
- assert(main_channel);
- return true;
+ return ret;
}
void migration_ioc_process_incoming(QIOChannel *ioc, Error **errp)
@@ -948,13 +957,12 @@ void migration_ioc_process_incoming(QIOChannel *ioc, Error **errp)
MigrationIncomingState *mis = migration_incoming_get_current();
Error *local_err = NULL;
QEMUFile *f;
- bool default_channel = true;
uint32_t channel_magic = 0;
+ uint8_t channel = CH_DEFAULT;
int ret = 0;
- if (migrate_multifd() && !migrate_mapped_ram() &&
- !migrate_postcopy_ram() &&
- qio_channel_has_feature(ioc, QIO_CHANNEL_FEATURE_READ_MSG_PEEK)) {
+ if (!migration_should_start_incoming(channel)
+ && qio_channel_has_feature(ioc, QIO_CHANNEL_FEATURE_READ_MSG_PEEK)) {
/*
* With multiple channels, it is possible that we receive channels
* out of order on destination side, causing incorrect mapping of
@@ -972,35 +980,46 @@ void migration_ioc_process_incoming(QIOChannel *ioc, Error **errp)
return;
}
- default_channel = (channel_magic == cpu_to_be32(QEMU_VM_FILE_MAGIC));
+ if (channel_magic == cpu_to_be32(QEMU_VM_FILE_MAGIC)) {
+ channel = CH_DEFAULT;
+ } else if (channel_magic == cpu_to_be32(MULTIFD_MAGIC)) {
+ channel = CH_MULTIFD;
+ } else {
+ error_report("%s: could not identify channel, unknown magic: %u",
+ __func__, channel_magic);
+ return;
+ }
+
} else {
- default_channel = !mis->from_src_file;
+ channel = CH_POSTCOPY;
}
if (multifd_recv_setup(errp) != 0) {
return;
}
- if (default_channel) {
+ if (channel == CH_DEFAULT) {
f = qemu_file_new_input(ioc);
migration_incoming_setup(f);
- } else {
+ } else if (channel == CH_MULTIFD) {
/* Multiple connections */
- assert(migration_needs_multiple_sockets());
if (migrate_multifd()) {
multifd_recv_new_channel(ioc, &local_err);
- } else {
+ }
+ if (local_err) {
+ error_propagate(errp, local_err);
+ return;
+ }
+ } else if (channel == CH_POSTCOPY) {
+ if (migrate_postcopy()) {
assert(migrate_postcopy_preempt());
+ assert(!mis->postcopy_qemufile_dst);
f = qemu_file_new_input(ioc);
postcopy_preempt_new_channel(mis, f);
}
- if (local_err) {
- error_propagate(errp, local_err);
- return;
- }
}
- if (migration_should_start_incoming(default_channel)) {
+ if (migration_should_start_incoming(channel)) {
/* If it's a recovery, we're done */
if (postcopy_try_recover()) {
return;
@@ -1017,21 +1036,22 @@ void migration_ioc_process_incoming(QIOChannel *ioc, Error **errp)
*/
bool migration_has_all_channels(void)
{
+ bool ret = false;
MigrationIncomingState *mis = migration_incoming_get_current();
if (!mis->from_src_file) {
- return false;
+ return ret;
}
if (migrate_multifd()) {
- return multifd_recv_all_channels_created();
+ ret = multifd_recv_all_channels_created();
}
- if (migrate_postcopy_preempt()) {
- return mis->postcopy_qemufile_dst != NULL;
+ if (ret && migrate_postcopy_preempt()) {
+ ret = mis->postcopy_qemufile_dst != NULL;
}
- return true;
+ return ret;
}
int migrate_send_rp_switchover_ack(MigrationIncomingState *mis)
@@ -3239,6 +3259,11 @@ static MigIterateState migration_iteration_run(MigrationState *s)
/* Still a significant amount to transfer */
if (!in_postcopy && must_precopy <= s->threshold_size && can_switchover &&
qatomic_read(&s->start_postcopy)) {
+
+ if (migrate_multifd()) {
+ multifd_send_sync_main();
+ }
+
if (postcopy_start(s, &local_err)) {
migrate_set_error(s, local_err);
error_report_err(local_err);