[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v14 13/13] Add XBZRLE statistics
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v14 13/13] Add XBZRLE statistics |
Date: |
Tue, 03 Jul 2012 19:35:28 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120615 Thunderbird/13.0.1 |
On 07/03/2012 07:52 AM, Orit Wasserman wrote:
> Signed-off-by: Benoit Hudzia <address@hidden>
> Signed-off-by: Petter Svard <address@hidden>
> Signed-off-by: Aidan Shribman <address@hidden>
> Signed-off-by: Orit Wasserman <address@hidden>
>
> ##
> # @MigrationInfo
> @@ -282,13 +307,18 @@
> # status, only returned if status is 'active' and it is a block
> # migration
> #
> -# Since: 0.14.0
> +# @params: #optional a list describing all the migration capabilities state
s/@params/@capabilities/ to match the schema. also, since
'*capabilities' was added in an earlier patch, this portion of the
documentation should be rebased into that same patch.
> +++ b/qmp-commands.hx
> @@ -2093,18 +2093,28 @@ The main json-object contains the following:
> - "transferred": amount transferred (json-int)
> - "remaining": amount remaining (json-int)
> - "total": total (json-int)
> + - "duplicate": number of duplicated pages (json-int)
> + - "normal" : number of normal pages transferred (json-int)
> - "disk": only present if "status" is "active" and it is a block migration,
> it is a json-object with the following disk information (in bytes):
> - "transferred": amount transferred (json-int)
> - "remaining": amount remaining (json-int)
> - "total": total (json-int)
> +- "params": migration capabilites state
s/params/capabilities/, s/capabilites/capabilities/
> + - "xbzrle" : on/off (json-bool)
on/off is not a json-bool; furthermore, MigrationCapabilityInfo (patch
2/13) is documented as being { "capability":"xbzrle", "state":true }
rather than { "xbzrle":"on" }
> 1. Before the first migration
> -
> -> { "execute": "query-migrate" }
> -<- { "return": {} }
> +<- { "return": { "params" : { "xbzrle" : "off" } } }
which means this example does not match your code
>
> 2. Migration is done and has succeeded
>
> @@ -2122,6 +2132,7 @@ Examples:
> <- {
> "return":{
> "status":"active",
> + "params" : { "xbzrle" : "off" },
neither does this. Should be:
"capabilities": [ { "capability":"xbzrle", "state":false } ],
> @@ -2136,6 +2147,7 @@ Examples:
> <- {
> "return":{
> "status":"active",
> + "params" : { "xbzrle" : "off" },
and again
> "ram":{
> "total":1057024,
> "remaining":1053304,
> @@ -2149,6 +2161,28 @@ Examples:
> }
> }
>
> +5. Migration is being performed and XBZRLE is active:
> +
> +-> { "execute": "query-migrate" }
> +<- {
> + "return":{
> + "status":"active",
> + "params" : { "xbzrle" : "on" },
and again
--
Eric Blake address@hidden +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
[Qemu-devel] [PATCH v14 05/13] Add uleb encoding/decoding functions, Orit Wasserman, 2012/07/03
[Qemu-devel] [PATCH v14 11/13] Add XBZRLE to ram_save_block and ram_save_live, Orit Wasserman, 2012/07/03
[Qemu-devel] [PATCH v14 02/13] Add migration capabilities, Orit Wasserman, 2012/07/03
[Qemu-devel] [PATCH v14 13/13] Add XBZRLE statistics, Orit Wasserman, 2012/07/03
- Re: [Qemu-devel] [PATCH v14 13/13] Add XBZRLE statistics,
Eric Blake <=
[Qemu-devel] [PATCH v14 12/13] Add set_cachesize command, Orit Wasserman, 2012/07/03
[Qemu-devel] [PATCH v14 09/13] Add migration_end function, Orit Wasserman, 2012/07/03