Berto,
Well noted about the "after implementation".
Kindly,
Andrey Shinkevich
On 05.11.2018 19:08, Alberto Garcia wrote:
> On Wed 31 Oct 2018 05:47:19 PM CET, Andrey Shinkevich
> <andrey.shinkev...@virtuozzo.com> wrote:
>> Adding a parameter to QMP block-stream command to allow discarding
>> blocks in the backing chain while blocks are being copied to the
>> active layer.
>>
>> Signed-off-by: Andrey Shinkevich <andrey.shinkev...@virtuozzo.com>
> I haven't checked the other patch yet, but if you're going to add new
> API (this new 'discard' parameter) I suggest you to do it _after_ the
> implementation.
>
>> --- a/qapi/block-core.json
>> +++ b/qapi/block-core.json
>> @@ -2329,6 +2329,9 @@
>> #
>> # @speed: the maximum speed, in bytes per second
>> #
>> +# @discard: true to delete blocks duplicated in old backing files.
>> +# (default: false). Since 3.1.
>> +#
>> # @on-error: the action to take on an error (default report).
>> # 'stop' and 'enospc' can only be used if the block device
>> # supports io-status (see BlockInfo). Since 1.3.
>> @@ -2361,7 +2364,7 @@
>> { 'command': 'block-stream',
>> 'data': { '*job-id': 'str', 'device': 'str', '*base': 'str',
>> '*base-node': 'str', '*backing-file': 'str', '*speed': 'int',
>> - '*on-error': 'BlockdevOnError',
>> + '*discard': 'bool', '*on-error': 'BlockdevOnError',
>> '*auto-finalize': 'bool', '*auto-dismiss': 'bool' } }
> Berto
