Re: [Qemu-devel] [PATCH v3 0/3] qapi: block-dirty-bitmap-remove transaction action

[Vladimir Sementsov-Ogievskiy]

15.07.2019 22:48, John Snow wrote:
> 
> 
> On 7/8/19 6:04 PM, John Snow wrote:
>> Hi, this is a proposal based off of Vladimir's patchset:
>> [Qemu-devel] [PATCH 0/4] qapi: block-dirty-bitmap-remove transaction action
>>
>> ===
>> V3:
>> ===
>>
>> 001/3:[----] [--] 'blockdev: reduce aio_context locked sections in bitmap 
>> add/remove'
>> 002/3:[0024] [FC] 'qapi: implement block-dirty-bitmap-remove transaction 
>> action'
>> 003/3:[----] [--] 'iotests: test bitmap moving inside 254'
>>
>> - Changed "squelch_persistence" to "skip_store"
>> - Use Max's suggestion for return expr
>>
>> ===
>> V2:
>> ===
>>
>> It replaces patches two and three with a modified patch (now patch 2)
>> that foregoes the need for a hide()/unhide() bitmap API. I think it's
>> suitable as a smaller alternative, but I'm not sure if it covers all
>> of the use cases of the original series.
>>
>> Patches 1 and 3 (formerly 4) included as-is.
>>
>> John Snow (1):
>>    qapi: implement block-dirty-bitmap-remove transaction action
>>
>> Vladimir Sementsov-Ogievskiy (2):
>>    blockdev: reduce aio_context locked sections in bitmap add/remove
>>    iotests: test bitmap moving inside 254
>>
>>   block.c                        |   2 +-
>>   block/dirty-bitmap.c           |  15 +++--
>>   blockdev.c                     | 105 ++++++++++++++++++++++++++-------
>>   include/block/dirty-bitmap.h   |   2 +-
>>   migration/block-dirty-bitmap.c |   2 +-
>>   qapi/transaction.json          |   2 +
>>   tests/qemu-iotests/254         |  30 +++++++++-
>>   tests/qemu-iotests/254.out     |  82 +++++++++++++++++++++++++
>>   8 files changed, 206 insertions(+), 34 deletions(-)
>>
> 
> Thanks, applied to my bitmaps tree:
> 
> https://github.com/jnsnow/qemu/commits/bitmaps
> https://github.com/jnsnow/qemu.git
> 
> --js
> 
> 
> (Vladimir: if this isn't amenable to you, it's going in for 4.2, so we
> have until the next freeze to change it. Let me know, OK?)
> 


And finally I'm here :)

Thanks a lot for doing this job and for your explanations in other threads which
I'm reading today and sorry for the delay! I'll look through these series soon.

Actually, my second child (girl:) was born a month ago, and then her elder 
brother
was ill, so I took two weeks sick leave after about two weeks vacation and 
forget
about work for a month.

Hmm. And Nikolay, who doing libvirt part is on vocation now (I started bitmap 
remove
transaction series by his request), so I don't know about the end of the story 
with
release and this functionality..

Anyway, it's cool, thanks!

-- 
Best regards,
Vladimir