On 06/25/2018 12:57 PM, Vladimir Sementsov-Ogievskiy wrote:
> Vladimir Sementsov-Ogievskiy (2):
> block/dirty-bitmap: add bdrv_enable_dirty_bitmap_locked
> dirty-bitmap: fix double lock on bitmap enabling
>
> include/block/dirty-bitmap.h | 1 +
> block/dirty-bitmap.c | 12 +++++++++---
> migration/block-dirty-bitmap.c | 4 ++--
> 3 files changed, 12 insertions(+), 5 deletions(-)
>
ACK,
though it looks really odd that we've got enable/disable but only one
corresponding _locked version.
I have a patch somewhere in my local files to create a combined
_set_disabled() that would cut the number of functions we have by a
little bit.
I'll take this first though, as an iotest fix.
--js