[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 1/6] multifd: Create property multifd-sync-after-each-section
From: |
Juan Quintela |
Subject: |
[PATCH v2 1/6] multifd: Create property multifd-sync-after-each-section |
Date: |
Thu, 28 Jul 2022 13:59:52 +0200 |
We used to synchronize all channels at the end of each RAM section
sent. That is not needed, so preparing to only synchronize once every
full round in latests patches.
Notice that we initialize the property as true. We will change the
default when we introduce the new mechanism.
Signed-off-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
---
Rename each-iteration to after-each-section
Signed-off-by: Juan Quintela <quintela@redhat.com>
---
migration/migration.h | 6 ++++++
hw/core/machine.c | 1 +
migration/migration.c | 11 ++++++++++-
3 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/migration/migration.h b/migration/migration.h
index cdad8aceaa..6abd2a51f5 100644
--- a/migration/migration.h
+++ b/migration/migration.h
@@ -373,6 +373,11 @@ struct MigrationState {
* This save hostname when out-going migration starts
*/
char *hostname;
+ /*
+ * Synchronize channels after each section is sent.
+ * We used to do that on the past, but it is suboptimal.
+ */
+ bool multifd_sync_after_each_section;
};
void migrate_set_state(int *state, int old_state, int new_state);
@@ -415,6 +420,7 @@ int migrate_multifd_channels(void);
MultiFDCompression migrate_multifd_compression(void);
int migrate_multifd_zlib_level(void);
int migrate_multifd_zstd_level(void);
+bool migrate_multifd_sync_after_each_section(void);
#ifdef CONFIG_LINUX
bool migrate_use_zero_copy_send(void);
diff --git a/hw/core/machine.c b/hw/core/machine.c
index a673302cce..9645a25f8f 100644
--- a/hw/core/machine.c
+++ b/hw/core/machine.c
@@ -43,6 +43,7 @@
GlobalProperty hw_compat_7_0[] = {
{ "arm-gicv3-common", "force-8-bit-prio", "on" },
{ "nvme-ns", "eui64-default", "on"},
+ { "migration", "multifd-sync-after-each-section", "on"},
};
const size_t hw_compat_7_0_len = G_N_ELEMENTS(hw_compat_7_0);
diff --git a/migration/migration.c b/migration/migration.c
index e03f698a3c..ebca4f2d8a 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -2592,6 +2592,13 @@ bool migrate_use_multifd(void)
return s->enabled_capabilities[MIGRATION_CAPABILITY_MULTIFD];
}
+bool migrate_multifd_sync_after_each_section(void)
+{
+ MigrationState *s = migrate_get_current();
+
+ return s->multifd_sync_after_each_section;
+}
+
bool migrate_pause_before_switchover(void)
{
MigrationState *s;
@@ -4384,7 +4391,9 @@ static Property migration_properties[] = {
DEFINE_PROP_STRING("tls-creds", MigrationState, parameters.tls_creds),
DEFINE_PROP_STRING("tls-hostname", MigrationState,
parameters.tls_hostname),
DEFINE_PROP_STRING("tls-authz", MigrationState, parameters.tls_authz),
-
+ /* We will change to false when we introduce the new mechanism */
+ DEFINE_PROP_BOOL("multifd-sync-after-each-section", MigrationState,
+ multifd_sync_after_each_section, true),
/* Migration capabilities */
DEFINE_PROP_MIG_CAP("x-xbzrle", MIGRATION_CAPABILITY_XBZRLE),
DEFINE_PROP_MIG_CAP("x-rdma-pin-all", MIGRATION_CAPABILITY_RDMA_PIN_ALL),
--
2.37.1
- [PATCH v2 0/6] Eliminate multifd flush, Juan Quintela, 2022/07/28
- [PATCH v2 1/6] multifd: Create property multifd-sync-after-each-section,
Juan Quintela <=
- [PATCH v2 3/6] migration: Simplify ram_find_and_save_block(), Juan Quintela, 2022/07/28
- [PATCH v2 2/6] multifd: Protect multifd_send_sync_main() calls, Juan Quintela, 2022/07/28
- [PATCH v2 6/6] ram: Document migration ram flags, Juan Quintela, 2022/07/28
- [PATCH v2 5/6] multifd: Only sync once each full round of memory, Juan Quintela, 2022/07/28
- [PATCH v2 4/6] migration: Make find_dirty_block() return a single parameter, Juan Quintela, 2022/07/28