[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 29/57] migrate_start_postcopy: Command to trigger
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PULL 29/57] migrate_start_postcopy: Command to trigger transition to postcopy |
Date: |
Mon, 9 Nov 2015 15:19:49 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
[adding Markus for a qapi question]
On 11/09/2015 10:28 AM, Juan Quintela wrote:
> From: "Dr. David Alan Gilbert" <address@hidden>
>
> Once postcopy is enabled (with migrate_set_capability), the migration
> will still start on precopy mode. To cause a transition into postcopy
> the:
>
> migrate_start_postcopy
>
> command must be issued. Postcopy will start sometime after this
> (when it's next checked in the migration loop).
>
> Issuing the command before migration has started will error,
> and issuing after it has finished is ignored.
>
> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> Reviewed-by: Juan Quintela <address@hidden>
> Reviewed-by: Amit Shah <address@hidden>
> Signed-off-by: Juan Quintela <address@hidden>
> ---
I know I reviewed an earlier version of this patch, but that was
probably before 24/57 of this pull request spelled the migration
capability bit as x-postcopy-ram.
> +++ b/qapi-schema.json
> @@ -702,6 +702,14 @@
> '*tls-port': 'int', '*cert-subject': 'str' } }
>
> ##
> +# @migrate-start-postcopy
> +#
> +# Switch migration to postcopy mode
No documentation on the relation to the [x-]postcopy-ram capability bit?
Will this command always fail if that bit is not set?
> +#
> +# Since: 2.5
> +{ 'command': 'migrate-start-postcopy' }
Should we rename this command to 'x-migrate-start-postcopy' until we are
ready to rename the entire feature to the stable namespace?
If so, I'm okay with that as a followup patch (so as not to delay the
pull request), but we should really make up our minds what 2.5 will
provide on this front.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PULL 22/57] Return path: Source handling of return path, (continued)
- [Qemu-devel] [PULL 22/57] Return path: Source handling of return path, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 25/57] Add wrappers and handlers for sending/receiving the postcopy-ram migration messages., Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 26/57] MIG_CMD_PACKAGED: Send a packaged chunk of migration stream, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 23/57] Rework loadvm path for subloops, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 27/57] Modify save_live_pending for postcopy, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 31/57] MIGRATION_STATUS_POSTCOPY_ACTIVE: Add new migration state, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 28/57] postcopy: OS support test, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 30/57] migration_completion: Take current state, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 32/57] Avoid sending vmdescription during postcopy, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 29/57] migrate_start_postcopy: Command to trigger transition to postcopy, Juan Quintela, 2015/11/09
- Re: [Qemu-devel] [PULL 29/57] migrate_start_postcopy: Command to trigger transition to postcopy,
Eric Blake <=
- [Qemu-devel] [PULL 33/57] Add qemu_savevm_state_complete_postcopy, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 35/57] migration_completion: Take current state, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 36/57] postcopy: Incoming initialisation, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 34/57] Postcopy: Maintain unsentmap, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 38/57] Postcopy: Postcopy startup in migration thread, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 37/57] postcopy: ram_enable_notify to switch on userfault, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 39/57] Postcopy: End of iteration, Juan Quintela, 2015/11/09
- [Qemu-devel] [PULL 40/57] Page request: Add MIG_RP_MSG_REQ_PAGES reverse command, Juan Quintela, 2015/11/09