[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v15 11/14] migration: modify migration_channels_and_uri_compatibl
From: |
Fabiano Rosas |
Subject: |
[PATCH v15 11/14] migration: modify migration_channels_and_uri_compatible() for new QAPI syntax |
Date: |
Mon, 23 Oct 2023 15:20:50 -0300 |
From: Het Gala <het.gala@nutanix.com>
migration_channels_and_uri_compatible() check for transport mechanism
suitable for multifd migration gets executed when the caller calls old
uri syntax. It needs it to be run when using the modern MigrateChannel
QAPI syntax too.
After URI -> 'MigrateChannel' :
migration_channels_and_uri_compatible() ->
migration_channels_and_transport_compatible() passes object as argument
and check for valid transport mechanism.
Suggested-by: Aravind Retnakaran <aravind.retnakaran@nutanix.com>
Signed-off-by: Het Gala <het.gala@nutanix.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Fabiano Rosas <farosas@suse.de>
---
migration/migration.c | 33 ++++++++++++++++++---------------
1 file changed, 18 insertions(+), 15 deletions(-)
diff --git a/migration/migration.c b/migration/migration.c
index 4356d9e701..192b7a97be 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -107,17 +107,20 @@ static bool migration_needs_multiple_sockets(void)
return migrate_multifd() || migrate_postcopy_preempt();
}
-static bool uri_supports_multi_channels(const char *uri)
+static bool transport_supports_multi_channels(SocketAddress *saddr)
{
- return strstart(uri, "tcp:", NULL) || strstart(uri, "unix:", NULL) ||
- strstart(uri, "vsock:", NULL);
+ return saddr->type == SOCKET_ADDRESS_TYPE_INET ||
+ saddr->type == SOCKET_ADDRESS_TYPE_UNIX ||
+ saddr->type == SOCKET_ADDRESS_TYPE_VSOCK;
}
static bool
-migration_channels_and_uri_compatible(const char *uri, Error **errp)
+migration_channels_and_transport_compatible(MigrationAddress *addr,
+ Error **errp)
{
if (migration_needs_multiple_sockets() &&
- !uri_supports_multi_channels(uri)) {
+ (addr->transport == MIGRATION_ADDRESS_TYPE_SOCKET) &&
+ !transport_supports_multi_channels(&addr->u.socket)) {
error_setg(errp, "Migration requires multi-channel URIs (e.g. tcp)");
return false;
}
@@ -509,15 +512,15 @@ static void qemu_start_incoming_migration(const char
*uri, bool has_channels,
return;
}
- /* URI is not suitable for migration? */
- if (!migration_channels_and_uri_compatible(uri, errp)) {
- return;
- }
-
if (uri && !migrate_uri_parse(uri, &channel, errp)) {
return;
}
+ /* transport mechanism not suitable for migration? */
+ if (!migration_channels_and_transport_compatible(channel, errp)) {
+ return;
+ }
+
migrate_set_state(&mis->state, MIGRATION_STATUS_NONE,
MIGRATION_STATUS_SETUP);
@@ -1812,15 +1815,15 @@ void qmp_migrate(const char *uri, bool has_channels,
return;
}
- /* URI is not suitable for migration? */
- if (!migration_channels_and_uri_compatible(uri, errp)) {
- return;
- }
-
if (!migrate_uri_parse(uri, &channel, errp)) {
return;
}
+ /* transport mechanism not suitable for migration? */
+ if (!migration_channels_and_transport_compatible(channel, errp)) {
+ return;
+ }
+
resume_requested = has_resume && resume;
if (!migrate_prepare(s, has_blk && blk, has_inc && inc,
resume_requested, errp)) {
--
2.35.3
- Re: [PATCH v15 06/14] migration: convert rdma backend to accept MigrateAddress, (continued)
- [PATCH v15 07/14] migration: convert exec backend to accept MigrateAddress., Fabiano Rosas, 2023/10/23
- [PATCH v15 08/14] migration: Convert the file backend to the new QAPI syntax, Fabiano Rosas, 2023/10/23
- [PATCH v15 09/14] migration: New migrate and migrate-incoming argument 'channels', Fabiano Rosas, 2023/10/23
- [PATCH v15 10/14] fixup! migration: New migrate and migrate-incoming argument 'channels', Fabiano Rosas, 2023/10/23
- [PATCH v15 11/14] migration: modify migration_channels_and_uri_compatible() for new QAPI syntax,
Fabiano Rosas <=
- [PATCH v15 12/14] migration: Implement MigrateChannelList to qmp migration flow., Fabiano Rosas, 2023/10/23
- [PATCH v15 13/14] migration: Implement MigrateChannelList to hmp migration flow., Fabiano Rosas, 2023/10/23
- Re: [PATCH v15 13/14] migration: Implement MigrateChannelList to hmp migration flow., Peter Xu, 2023/10/31