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

[John Snow <jsnow@redhat.com>]

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?)