[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 27/35] postcopy/outgoing: implement forward/b
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v3 27/35] postcopy/outgoing: implement forward/backword prefault |
Date: |
Fri, 02 Nov 2012 09:22:09 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121016 Thunderbird/16.0.1 |
On 11/01/2012 11:24 PM, Isaku Yamahata wrote:
>>> +++ b/qapi-schema.json
>>> @@ -2095,7 +2095,8 @@
>>> ##
>>> { 'command': 'migrate',
>>> 'data': {'uri': 'str', '*blk': 'bool', '*inc': 'bool', '*detach': 'bool'
>>> ,
>>> - '*postcopy': 'bool', '*nobg': 'bool'} }
>>> + '*postcopy': 'bool', '*nobg': 'bool',
>>> + '*forward': 'int', '*backward': 'int'} }
>>
>> Do we really want to be adding new options to migrate (and if so,
>> where's the documentation), or do we need a new monitor command similar
>> to migrate-set-capabilities or migrate-set-cache-size?
>
> Okay, migrate-set-capabilities seems usable for boolean and scalable
> for future extension.
> On the other hand, migrate-set-cache-size takes only single integer
> as arguments. So it doesn't seem usable without modification.
> How about this?
>
> { 'type': 'MigrationParameters',
> 'data': {'parameter': 'name': 'str', 'value': 'int' } }
More like:
{ 'enum': 'MigrationParameterName',
'data': ['ParameterName'... ] }
{ 'type': 'MigrationParameter',
'data': {'parameter': 'MigrationParameterName', 'value': 'int' } }
>
> { 'command': 'migrate-set-parameters',
> 'data': { 'parameters' ['MigrationParameters']}}
Yes, this seems more extensible.
>
>
> { 'command': 'query-migrate-parameters',
> 'returns': [['MigrationParameters']]}
One layer too many of [], but yes, this also seems reasonable.
--
Eric Blake address@hidden +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature