qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

  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).