Re: [PATCH V5 15/23] migration: cpr-transfer mode

[Markus Armbruster <armbru@redhat.com>]

Steve Sistare <steven.sistare@oracle.com> writes:

> Add the cpr-transfer migration mode, which allows the user to transfer
> a guest to a new QEMU instance on the same host with minimal guest pause
> time, by preserving guest RAM in place, albeit with new virtual addresses
> in new QEMU, and by preserving device file descriptors.  Pages that were
> locked in memory for DMA in old QEMU remain locked in new QEMU, because the
> descriptor of the device that locked them remains open.
>
> cpr-transfer preserves memory and devices descriptors by sending them to
> new QEMU over a unix domain socket using SCM_RIGHTS.  Such CPR state cannot
> be sent over the normal migration channel, because devices and backends
> are created prior to reading the channel,

Is that an artifact of the way QEMU starts up, or is it fundamental?

>                                           so this mode sends CPR state
> over a second "cpr" migration channel.  New QEMU reads the cpr channel
> prior to creating devices or backends.  The user specifies the cpr channel
> in the channel arguments on the outgoing side, and in a second -incoming
> command-line parameter on the incoming side.
>
> The user must start old QEMU with the the '-machine aux-ram-share=on' option,
> which allows anonymous memory to be transferred in place to the new process
> by transferring a memory descriptor for each ram block.  Memory-backend
> objects must have the share=on attribute, but memory-backend-epc is not
> supported.
>
> The user starts new QEMU on the same host as old QEMU, with command-line
> arguments to create the same machine, plus the -incoming option for the
> main migration channel, like normal live migration.  In addition, the user
> adds a second -incoming option with channel type "cpr".  The CPR channel
> address must be a type, such as unix socket, that supports SCM_RIGHTS.
>
> To initiate CPR, the user issues a migrate command to old QEMU, adding
> a second migration channel of type "cpr" in the channels argument.
> Old QEMU stops the VM, saves state to the migration channels, and enters
> the postmigrate state.  New QEMU mmap's memory descriptors, and execution
> resumes.
>
> The implementation splits qmp_migrate into start and finish functions.
> Start sends CPR state to new QEMU, which responds by closing the CPR
> channel.  Old QEMU detects the HUP then calls finish, which connects the
> main migration channel.
>
> In summary, the usage is:
>
>   qemu-system-$arch -machine aux-ram-share=on ...
>
>   start new QEMU with "-incoming <main-uri> -incoming <cpr-channel>"
>
>   Issue commands to old QEMU:
>     migrate_set_parameter mode cpr-transfer
>
>     {"execute": "migrate", ...
>         {"channel-type": "main"...}, {"channel-type": "cpr"...} ... }
>
> Signed-off-by: Steve Sistare <steven.sistare@oracle.com>

[...]

> diff --git a/migration/vmstate-types.c b/migration/vmstate-types.c
> index f31deb3..2210f0c 100644
> --- a/migration/vmstate-types.c
> +++ b/migration/vmstate-types.c
> @@ -15,6 +15,7 @@
>  #include "qemu-file.h"
>  #include "migration.h"
>  #include "migration/vmstate.h"
> +#include "migration/client-options.h"
>  #include "qemu/error-report.h"
>  #include "qemu/queue.h"
>  #include "trace.h"
> diff --git a/qapi/migration.json b/qapi/migration.json
> index a605dc2..35309dc 100644
> --- a/qapi/migration.json
> +++ b/qapi/migration.json
> @@ -614,9 +614,47 @@
>  #     or COLO.
>  #
>  #     (since 8.2)
> +#
> +# @cpr-transfer: This mode allows the user to transfer a guest to a
> +#     new QEMU instance on the same host with minimal guest pause
> +#     time, by preserving guest RAM in place, albeit with new virtual
> +#     addresses in new QEMU.  Devices and their pinned pages will also
> +#     be preserved in a future QEMU release.

Maybe "@cpr-transfer: Checkpoint and restart migration mode minimizes
guest pause time by transferring guest RAM without copying it."

If you want to mention the guest RAM mapping differs between old and new
QEMU, that's fine, but it's also detail, so I'd do it further down.

> +#
> +#     The user starts new QEMU on the same host as old QEMU, with
> +#     command-line arguments to create the same machine, plus the
> +#     -incoming option for the main migration channel, like normal
> +#     live migration.  In addition, the user adds a second -incoming
> +#     option with channel type "cpr".  The CPR channel address must
> +#     be a type, such as unix socket, that supports SCM_RIGHTS.

Permit me to indulge in a bit of pedantry...  A channel address doesn't
support SCM_RIGHTS, only a channel may.  A channel supports it when it
is backed by a UNIX domain socket.  The channel's socket's transport
type need not be 'unix' for that, it could also be 'fd'.

Suggest something like "This CPR channel must be a UNIX domain socket."

If you want to say why, that's fine: "This CPR channel must support file
descriptor transfer, i.e. it must be a UNIX domain socket."

If you want to mention SCM_RIGHTS, that's fine, too: "This CPR channel
must support file descriptor transfer with SCM_RIGHTS, i.e. it must be a
UNIX domain socket."

> +#
> +#     To initiate CPR, the user issues a migrate command to old QEMU,
> +#     adding a second migration channel of type "cpr" in the channels

in the channel's

> +#     argument.  Old QEMU stops the VM, saves state to the migration
> +#     channels, and enters the postmigrate state.  Execution resumes
> +#     in new QEMU.
> +#
> +#     New QEMU reads the CPR channel before opening a monitor, hence
> +#     the CPR channel cannot be specified in the list of channels for
> +#     a migrate-incoming command.  It may only be specified on the
> +#     command line.

This is a restriction that could conceivably be lifted in the future,
right?

What happens if a user tries to specify it with migrate-incoming?  Fails
cleanly?  What's the error message?

Maybe simply:

         Currently, the CPR channel can only be specified on the command
         line, not with the migrate-incoming command.

with a big, fat comment explaining the restriction next to the spot
that reports the error.

> +#
> +#     The main channel address cannot be a file type, and for the tcp
> +#     type, the port cannot be 0 (meaning dynamically choose a port).

What's "the tcp type"?  URI "tcp:..." / channel
addr.transport=socket,addr.type=inet?

> +#
> +#     Memory-backend objects must have the share=on attribute, but
> +#     memory-backend-epc is not supported.  The VM must be started
> +#     with the '-machine aux-ram-share=on' option.
> +#
> +#     When using -incoming defer, you must issue the migrate command
> +#     to old QEMU before issuing any monitor commands to new QEMU.
> +#     However, new QEMU does not open and read the migration stream
> +#     until you issue the migrate incoming command.

I think some (all?) instances of "old QEMU" and "new QEMU" would read
better as "the old QEMU" and "the new QEMU".

> +#
> +#     (since 10.0)
>  ##
>  { 'enum': 'MigMode',
> -  'data': [ 'normal', 'cpr-reboot' ] }
> +  'data': [ 'normal', 'cpr-reboot', 'cpr-transfer' ] }
>  
>  ##
>  # @ZeroPageDetection:
> @@ -1578,11 +1616,12 @@
>  # The migration channel-type request options.
>  #
>  # @main: Main outbound migration channel.
> +# @cpr: Checkpoint and restart state channel.
>  #
>  # Since: 8.1
>  ##
>  { 'enum': 'MigrationChannelType',
> -  'data': [ 'main' ] }
> +  'data': [ 'main', 'cpr' ] }
>  
>  ##
>  # @MigrationChannel: