From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
To: zhanghailiang <zhang.zhanghailiang@huawei.com>
Cc: amit.shah@redhat.com, peter.huangpeng@huawei.com,
qemu-devel@nongnu.org, quintela@redhat.com
Subject: Re: [Qemu-devel] [PATCH 6/6] qmp-commands.hx: Document the missing options for migration capability commands
Date: Thu, 28 Jan 2016 18:24:25 +0000 [thread overview]
Message-ID: <20160128182424.GH2911@work-vm> (raw)
In-Reply-To: <1452829066-9764-7-git-send-email-zhang.zhanghailiang@huawei.com>
* zhanghailiang (zhang.zhanghailiang@huawei.com) wrote:
> Add the missing descriptions for the options of migration capability commands,
> and fix the example for query-migrate-capabilities command.
>
> Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> qmp-commands.hx | 15 ++++++++++++++-
> 1 file changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/qmp-commands.hx b/qmp-commands.hx
> index 0a2cded..12be357 100644
> --- a/qmp-commands.hx
> +++ b/qmp-commands.hx
> @@ -3647,7 +3647,9 @@ Enable/Disable migration capabilities
> - "rdma-pin-all": pin all pages when using RDMA during migration
> - "auto-converge": throttle down guest to help convergence of migration
> - "zero-blocks": compress zero blocks during block migration
> +- "compress": use multiple compression threads to accelerate live migration
> - "events": generate events for each migration state change
> +- "x-postcopy-ram": postcopy mode for live migration
>
> Arguments:
>
> @@ -3675,13 +3677,24 @@ Query current migration capabilities
> - "rdma-pin-all" : RDMA Pin Page state (json-bool)
> - "auto-converge" : Auto Converge state (json-bool)
> - "zero-blocks" : Zero Blocks state (json-bool)
> + - "compress": Multiple compression threads state (json-bool)
> + - "events": Migration state change event state (json-bool)
> + - "x-postcopy-ram": postcopy ram state (json-bool)
>
> Arguments:
>
> Example:
>
> -> { "execute": "query-migrate-capabilities" }
> -<- { "return": [ { "state": false, "capability": "xbzrle" } ] }
> +<- {"return": [
> + {"state": false, "capability": "xbzrle"},
> + {"state": false, "capability": "rdma-pin-all"},
> + {"state": false, "capability": "auto-converge"},
> + {"state": false, "capability": "zero-blocks"},
> + {"state": false, "capability": "compress"},
> + {"state": true, "capability": "events"},
> + {"state": false, "capability": "x-postcopy-ram"}
> + ]}
>
> EQMP
>
> --
> 1.8.3.1
>
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
next prev parent reply other threads:[~2016-01-28 18:24 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-15 3:37 [Qemu-devel] [PATCH 0/6] Some improvements and small fixes for migration zhanghailiang
2016-01-15 3:37 ` [Qemu-devel] [PATCH 1/6] ram: Split host_from_stream_offset() into two helper functions zhanghailiang
2016-01-15 3:37 ` [Qemu-devel] [PATCH 2/6] migration: Rename the'file' member of MigrationState zhanghailiang
2016-02-04 10:47 ` Amit Shah
2016-02-04 10:50 ` Dr. David Alan Gilbert
2016-02-04 10:57 ` Amit Shah
2016-02-04 11:10 ` Dr. David Alan Gilbert
2016-02-04 12:02 ` Hailiang Zhang
2016-01-15 3:37 ` [Qemu-devel] [PATCH 3/6] savevm: Split load vm state function qemu_loadvm_state zhanghailiang
2016-01-15 3:37 ` [Qemu-devel] [PATCH 4/6] migration/ram: Fix some helper functions' parameter to use PageSearchStatus zhanghailiang
2016-01-15 3:37 ` [Qemu-devel] [PATCH 5/6] qmp-commands.hx: Fix the missing options for migration parameters commands zhanghailiang
2016-01-28 18:22 ` Dr. David Alan Gilbert
2016-02-04 10:30 ` Amit Shah
2016-01-15 3:37 ` [Qemu-devel] [PATCH 6/6] qmp-commands.hx: Document the missing options for migration capability commands zhanghailiang
2016-01-28 18:24 ` Dr. David Alan Gilbert [this message]
2016-01-22 8:43 ` [Qemu-devel] [PATCH 0/6] Some improvements and small fixes for migration Hailiang Zhang
2016-01-30 10:32 ` Hailiang Zhang
2016-02-04 10:08 ` Amit Shah
2016-02-04 11:08 ` Amit Shah
2016-02-04 12:01 ` Hailiang Zhang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160128182424.GH2911@work-vm \
--to=dgilbert@redhat.com \
--cc=amit.shah@redhat.com \
--cc=peter.huangpeng@huawei.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
--cc=zhang.zhanghailiang@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).