[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v12 2/3] qcow2: Add list of bitmaps to ImageInfo
From: |
Andrey Shinkevich |
Subject: |
Re: [Qemu-block] [PATCH v12 2/3] qcow2: Add list of bitmaps to ImageInfoSpecificQCow2 |
Date: |
Wed, 6 Feb 2019 10:25:19 +0000 |
On 05/02/2019 23:33, Eric Blake wrote:
> On 2/5/19 2:14 PM, Andrey Shinkevich wrote:
>> In the 'Format specific information' section of the 'qemu-img info'
>> command output, the supplemental information about existing QCOW2
>> bitmaps will be shown, such as a bitmap name, flags and granularity:
>>
>
>> +##
>> +# @Qcow2BitmapInfo:
>> +#
>> +# Qcow2 bitmap information.
>> +#
>> +# @name: the name of the bitmap
>> +#
>> +# @granularity: granularity of the bitmap in bytes
>> +#
>> +# @flags: flags of the bitmap
>> +#
>> +# Since: 4.0
>
> You got rid of the docs for unknown-flags,
>
>> +##
>> +{ 'struct': 'Qcow2BitmapInfo',
>> + 'data': {'name': 'str', 'granularity': 'uint32',
>> + 'flags': ['Qcow2BitmapInfoFlags'],
>> + '*unknown-flags': 'uint32' } }
>
> but forgot to actually get rid of the field. That's a pretty simple
> cleanup, which I don't mind making if this series is otherwise ready to go.
>
> Reviewed-by: Eric Blake <address@hidden>
>
Dear Eric,
Thank you for being mindful of this series and for your
contribution to the common work!
--
With the best regards,
Andrey Shinkevich