[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v16 08/22] qcow2: add bitmaps exten
From: |
John Snow |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v16 08/22] qcow2: add bitmaps extension |
Date: |
Mon, 27 Feb 2017 18:09:20 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 |
On 02/27/2017 07:27 AM, Max Reitz wrote:
> On 25.02.2017 18:07, Vladimir Sementsov-Ogievskiy wrote:
>> Add bitmap extension as specified in docs/specs/qcow2.txt.
>> For now, just mirror extension header into Qcow2 state and check
>> constraints. Also, calculate refcounts for qcow2 bitmaps, to not break
>> qemu-img check.
>>
>> For now, disable image resize if it has bitmaps. It will be fixed later.
>>
>> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
>> ---
>> block/Makefile.objs | 2 +-
>> block/qcow2-bitmap.c | 439
>> +++++++++++++++++++++++++++++++++++++++++++++++++
>> block/qcow2-refcount.c | 6 +
>> block/qcow2.c | 124 +++++++++++++-
>> block/qcow2.h | 27 +++
>> 5 files changed, 592 insertions(+), 6 deletions(-)
>> create mode 100644 block/qcow2-bitmap.c
>
> Somehow I have the feeling Kevin will find bad things in this patch, but
> since I have already approved of all of the previous patches this one is
> composed of and the changes on top of that look OK to me:
>
> Reviewed-by: Max Reitz <address@hidden>
>
Pretty much the same sentiment as Max.
The patchset is now organized a bit strangely, but I did R-B all the
component pieces before, and the changes look fine.
Reviewed-by: John Snow <address@hidden>
- [Qemu-block] [PATCH v16 07/22] qcow2-refcount: rename inc_refcounts() and make it public, (continued)
- [Qemu-block] [PATCH v16 07/22] qcow2-refcount: rename inc_refcounts() and make it public, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 06/22] block/dirty-bitmap: add deserialize_ones func, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 05/22] block: fix bdrv_dirty_bitmap_granularity signature, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 14/22] block: add bdrv_can_store_new_dirty_bitmap, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 17/22] qmp: add autoload parameter to block-dirty-bitmap-add, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 11/22] block: introduce persistent dirty bitmaps, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 16/22] qmp: add persistent flag to block-dirty-bitmap-add, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 08/22] qcow2: add bitmaps extension, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 10/22] block/dirty-bitmap: add autoload field to BdrvDirtyBitmap, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 02/22] specs/qcow2: do not use wording 'bitmap header', Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 15/22] qcow2: add .bdrv_can_store_new_dirty_bitmap, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 18/22] qmp: add x-debug-block-dirty-bitmap-sha256, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 09/22] qcow2: autoloading dirty bitmaps, Vladimir Sementsov-Ogievskiy, 2017/02/25
- [Qemu-block] [PATCH v16 13/22] qcow2: add persistent dirty bitmaps support, Vladimir Sementsov-Ogievskiy, 2017/02/25