[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v5 02/10] HBitmap: Introduce "meta" bitmap to tr
From: |
John Snow |
Subject: |
Re: [Qemu-block] [PATCH v5 02/10] HBitmap: Introduce "meta" bitmap to track bit changes |
Date: |
Wed, 13 Jul 2016 13:39:34 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 |
On 06/22/2016 11:22 AM, Max Reitz wrote:
> On 03.06.2016 06:32, Fam Zheng wrote:
>> Upon each bit toggle, the corresponding bit in the meta bitmap will be
>> set.
>>
>> Signed-off-by: Fam Zheng <address@hidden>
>> Reviewed-by: John Snow <address@hidden>
>> ---
>> block/dirty-bitmap.c | 2 +-
>> include/qemu/hbitmap.h | 17 +++++++++++++
>> util/hbitmap.c | 69
>> +++++++++++++++++++++++++++++++++++++++-----------
>> 3 files changed, 72 insertions(+), 16 deletions(-)
>>
>> diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c
>> index ec073ee..628b77c 100644
>> --- a/block/dirty-bitmap.c
>> +++ b/block/dirty-bitmap.c
>> @@ -231,7 +231,7 @@ static void
>> bdrv_do_release_matching_dirty_bitmap(BlockDriverState *bs,
>> BdrvDirtyBitmap *bm, *next;
>> QLIST_FOREACH_SAFE(bm, &bs->dirty_bitmaps, list, next) {
>> if ((!bitmap || bm == bitmap) && (!only_named || bm->name)) {
>> - assert(!bitmap->active_iterators);
>> + assert(!bm->active_iterators);
>
> Same comment as for v4, this should be squashed into the previous patch.
>
> Max
>
Ah, replied too soon on #1. Thanks :)
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-block] [PATCH v5 02/10] HBitmap: Introduce "meta" bitmap to track bit changes,
John Snow <=