[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/1] migrate-start-postcopy: Improve text
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/1] migrate-start-postcopy: Improve text |
Date: |
Thu, 12 Nov 2015 08:17:54 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 11/12/2015 04:34 AM, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <address@hidden>
>
> Improve the text in both the qapi-schema and hmp help to point out
> you need to set the postcopy-ram capability prior to issuing
> migrate-start-postcopy.
>
> Also fix the text of the migrate_start_postcopy error that
> deals with capabilities.
>
> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
> ---
> v2
> Updated text to Erik's suggestion
It's Eric, but you're not the first (nor the last) to make that typo.
> +++ b/qapi-schema.json
> @@ -706,7 +706,9 @@
> ##
> # @migrate-start-postcopy
> #
> -# Switch migration to postcopy mode
> +# Followup to a migration command to switch the migration to postcopy mode.
Maybe "Followup to the migrate command to switch the migration..."
But the wording you have also covers the case if we add something other
than just 'migrate' to start a migration, so I'm fine either way.
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature