Re: [Qemu-devel] [PATCH 10/11] Add XBZRLE statistics

[Eric Blake]

On 08/02/2012 06:44 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>
> Signed-off-by: Juan Quintela <address@hidden>
> ---


>  
> +6. Migration is being performed and XBZRLE is active:
> +
> +-> { "execute": "query-migrate" }
> +<- {
> +      "return":{
> +         "status":"active",
> +         "capabilities" : [ { "capability": "xbzrle", "state" : true } ],
> +         "ram":{
> +            "total":1057024,
> +            "remaining":1053304,
> +            "transferred":3720,
> +            "duplicate": 10,
> +            "normal": 3333
> +         },
> +         "xbzrle-cache":{
> +            "cache-size": 1024,

Is this value in bytes?  It should be.  Which means that this example is
bogus - 1024 is smaller than one page, but your caching code rounded to
page boundaries.  Sounds like you forgot a scaling factor somewhere.
Also, s/: /:/

-- 
Eric Blake   address@hidden    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature