[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/6] migration: Updated QAPI format for 'migrate' qemu mon
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v3 2/6] migration: Updated QAPI format for 'migrate' qemu monitor command |
Date: |
Fri, 10 Feb 2023 10:07:01 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Just a quick one on naming before I forget:
Het Gala <het.gala@nutanix.com> writes:
> Existing 'migrate' QAPI design enforces transport mechanism, ip address
> of destination interface and corresponding port number in the form
> of a unified string 'uri' parameter for initiating a migration stream.
> This scheme has a significant flaw in it - double encoding of existing
> URIs to extract migration info.
>
> The current patch maps QAPI uri design onto well defined MigrateChannel
> struct. This modified QAPI helps in preventing multi-level uri
> encodings ('uri' parameter is kept for backward compatibility).
>
> Reviewed-by: Eric Blake <eblake@redhat.com>
> Reviewed-by: Markus Armbruster <armbru@redhat.com>
> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> Suggested-by: Daniel P. Berrange <berrange@redhat.com>
> Suggested-by: Manish Mishra <manish.mishra@nutanix.com>
> Suggested-by: Aravind Retnakaran <aravind.retnakaran@nutanix.com>
> Signed-off-by: Het Gala <het.gala@nutanix.com>
> ---
> qapi/migration.json | 129 +++++++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 127 insertions(+), 2 deletions(-)
>
> diff --git a/qapi/migration.json b/qapi/migration.json
> index c84fa10e86..261a6770e7 100644
> --- a/qapi/migration.json
> +++ b/qapi/migration.json
> @@ -1449,12 +1449,106 @@
> ##
> { 'command': 'migrate-continue', 'data': {'state': 'MigrationStatus'} }
>
> +##
> +# @MigrateTransport:
> +#
> +# The supported communication transport mechanisms for migration
> +#
> +# @socket: Supported communication type between two devices for migration.
> +# Socket is able to cover all of 'tcp', 'unix', 'vsock' and
> +# 'fd' already
> +#
> +# @exec: Supported communication type to redirect migration stream into file.
> +#
> +# @rdma: Supported communication type to redirect rdma type migration stream.
> +#
> +# Since 8.0
> +##
> +{ 'enum': 'MigrateTransport',
> + 'data': ['socket', 'exec', 'rdma'] }
> +
> +##
> +# @MigrateSocketAddr:
> +#
> +# To support different type of socket.
> +#
> +# Since 8.0
> +##
> +{ 'struct': 'MigrateSocketAddr',
We tend to avoid abbreviations in QAPI schema names. For instance, it's
SocketAddress, not SocketAddr. Please use Address, not Addr, for
consistency.
> + 'data': {'data': 'SocketAddress' } }
> +
[...]
- [PATCH v3 0/6] migration: Modified 'migrate' QAPI command for migration, Het Gala, 2023/02/09
- [PATCH v3 1/6] migration: moved hmp_split_at_commma() helper func to qapi-util.c file, Het Gala, 2023/02/09
- [PATCH v3 2/6] migration: Updated QAPI format for 'migrate' qemu monitor command, Het Gala, 2023/02/09
- Re: [PATCH v3 2/6] migration: Updated QAPI format for 'migrate' qemu monitor command,
Markus Armbruster <=
- [PATCH v3 3/6] migration: HMP side changes for modified 'migrate' QAPI design, Het Gala, 2023/02/09
- [PATCH v3 5/6] migration: Modified 'migrate-incoming' QAPI and HMP side changes on the destination interface., Het Gala, 2023/02/09
- [PATCH v3 4/6] migration: Avoid multiple parsing of uri in migration code flow, Het Gala, 2023/02/09
- [PATCH v3 6/6] migration: Established connection for listener sockets on the dest interface, Het Gala, 2023/02/09