[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] For 1.2: Re: [PATCH 3/8] migration: move total_time fro
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] For 1.2: Re: [PATCH 3/8] migration: move total_time from ram stats to migration info |
Date: |
Wed, 22 Aug 2012 08:22:19 -0500 |
User-agent: |
Notmuch/0.13.2+93~ged93d79 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Eric Blake <address@hidden> writes:
> On 08/18/2012 05:17 AM, Juan Quintela wrote:
>> Signed-off-by: Juan Quintela <address@hidden>
>> ---
>> hmp.c | 4 ++--
>> migration.c | 6 +++---
>> qapi-schema.json | 14 +++++++-------
>> qmp-commands.hx | 6 +++---
>> 4 files changed, 15 insertions(+), 15 deletions(-)
>>
>
>> +++ b/qapi-schema.json
>> @@ -290,10 +290,6 @@
>> #
>> # @total: total amount of bytes involved in the migration process
>> #
>> -# @total-time: total amount of ms since migration started. If
>> -# migration has ended, it returns the total migration
>> -# time. (since 1.2)
>> -#
>> # @duplicate: number of duplicate pages (since 1.2)
>> #
>> # @normal : number of normal pages (since 1.2)
>> @@ -304,8 +300,7 @@
>> ##
>> { 'type': 'MigrationStats',
>> 'data': {'transferred': 'int', 'remaining': 'int', 'total': 'int' ,
>> - 'total-time': 'int', 'duplicate': 'int', 'normal': 'int',
>> - 'normal-bytes': 'int' } }
>> + 'duplicate': 'int', 'normal': 'int', 'normal-bytes': 'int' } }
>>
>> ##
>> # @XBZRLECacheStats
>> @@ -350,12 +345,17 @@
>> # migration statistics, only returned if XBZRLE feature is
>> on and
>> # status is 'active' or 'completed' (since 1.2)
>> #
>> +# @total-time: total amount of milliseconds since migration started.
>> +# If migration has ended, it returns the total migration
>> +# time. (since 1.2)
>> +#
>> # Since: 0.14.0
>> ##
>> { 'type': 'MigrationInfo',
>> 'data': {'*status': 'str', '*ram': 'MigrationStats',
>> '*disk': 'MigrationStats',
>> - '*xbzrle-cache': 'XBZRLECacheStats'} }
>> + '*xbzrle-cache': 'XBZRLECacheStats',
>> + 'total-time': 'int'} }
>
> Anthony - are you planning on taking this series for 1.2?
No. This is a new feature and we're past freeze.
> If we don't
> get this patch in on time, then taking this for 1.3 would result in
> changing released QMP interface (right now, there has been no release
> with the field in the wrong type).
Ack. We need to preserve compat with the 1.2 interface.
Regards,
Anthony Liguori
>
> --
> Eric Blake address@hidden +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
Re: [Qemu-devel] [PATCH 3/8] migration: move total_time from ram stats to migration info, Orit Wasserman, 2012/08/21
[Qemu-devel] [PATCH 4/8] migration: store end_time in a local variable, Juan Quintela, 2012/08/18
[Qemu-devel] [PATCH 6/8] migration: rename expected_time to expected_downtime, Juan Quintela, 2012/08/18
[Qemu-devel] [PATCH 7/8] migration: export migrate_get_current(), Juan Quintela, 2012/08/18
[Qemu-devel] [PATCH 5/8] migration: print total downtime for final phase of migration, Juan Quintela, 2012/08/18