[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH v10 03/12] block/dirty-bitmap: add _locked version o
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
[Qemu-block] [PATCH v10 03/12] block/dirty-bitmap: add _locked version of bdrv_reclaim_dirty_bitmap |
Date: |
Wed, 7 Feb 2018 18:58:28 +0300 |
Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
---
include/block/dirty-bitmap.h | 3 +++
block/dirty-bitmap.c | 28 ++++++++++++++++++----------
2 files changed, 21 insertions(+), 10 deletions(-)
diff --git a/include/block/dirty-bitmap.h b/include/block/dirty-bitmap.h
index 93e128f81b..b2e1fd913a 100644
--- a/include/block/dirty-bitmap.h
+++ b/include/block/dirty-bitmap.h
@@ -92,5 +92,8 @@ BdrvDirtyBitmap *bdrv_dirty_bitmap_next(BlockDriverState *bs,
BdrvDirtyBitmap *bitmap);
char *bdrv_dirty_bitmap_sha256(const BdrvDirtyBitmap *bitmap, Error **errp);
int64_t bdrv_dirty_bitmap_next_zero(BdrvDirtyBitmap *bitmap, uint64_t start);
+BdrvDirtyBitmap *bdrv_reclaim_dirty_bitmap_locked(BlockDriverState *bs,
+ BdrvDirtyBitmap *bitmap,
+ Error **errp);
#endif
diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c
index 75435f6c2f..ce00ff3474 100644
--- a/block/dirty-bitmap.c
+++ b/block/dirty-bitmap.c
@@ -320,17 +320,13 @@ BdrvDirtyBitmap
*bdrv_dirty_bitmap_abdicate(BlockDriverState *bs,
* In cases of failure where we can no longer safely delete the parent,
* we may wish to re-join the parent and child/successor.
* The merged parent will be un-frozen, but not explicitly re-enabled.
- * Called with BQL taken.
+ * Called within bdrv_dirty_bitmap_lock..unlock and with BQL taken.
*/
-BdrvDirtyBitmap *bdrv_reclaim_dirty_bitmap(BlockDriverState *bs,
- BdrvDirtyBitmap *parent,
- Error **errp)
+BdrvDirtyBitmap *bdrv_reclaim_dirty_bitmap_locked(BlockDriverState *bs,
+ BdrvDirtyBitmap *parent,
+ Error **errp)
{
- BdrvDirtyBitmap *successor;
-
- qemu_mutex_lock(parent->mutex);
-
- successor = parent->successor;
+ BdrvDirtyBitmap *successor = parent->successor;
if (!successor) {
error_setg(errp, "Cannot reclaim a successor when none is present");
@@ -344,9 +340,21 @@ BdrvDirtyBitmap
*bdrv_reclaim_dirty_bitmap(BlockDriverState *bs,
bdrv_release_dirty_bitmap_locked(bs, successor);
parent->successor = NULL;
+ return parent;
+}
+
+/* Called with BQL taken. */
+BdrvDirtyBitmap *bdrv_reclaim_dirty_bitmap(BlockDriverState *bs,
+ BdrvDirtyBitmap *parent,
+ Error **errp)
+{
+ BdrvDirtyBitmap *ret;
+
+ qemu_mutex_lock(parent->mutex);
+ ret = bdrv_reclaim_dirty_bitmap_locked(bs, parent, errp);
qemu_mutex_unlock(parent->mutex);
- return parent;
+ return ret;
}
/**
--
2.11.1
- [Qemu-block] [PATCH v10 00/12] Dirty bitmaps postcopy migration, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 07/12] migration: include migrate_dirty_bitmaps in migrate_postcopy, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 04/12] dirty-bitmap: add locked state, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 08/12] migration/qemu-file: add qemu_put_counted_string(), Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 12/12] iotests: add dirty bitmap postcopy test, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 05/12] migration: introduce postcopy-only pending, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 02/12] block/dirty-bitmap: fix locking in bdrv_reclaim_dirty_bitmap, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 01/12] block/dirty-bitmap: add bdrv_dirty_bitmap_enable_successor(), Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 09/12] migration: add is_active_iterate handler, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 03/12] block/dirty-bitmap: add _locked version of bdrv_reclaim_dirty_bitmap,
Vladimir Sementsov-Ogievskiy <=
- [Qemu-block] [PATCH v10 06/12] qapi: add dirty-bitmaps migration capability, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 11/12] iotests: add dirty bitmap migration test, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] [PATCH v10 10/12] migration: add postcopy migration of dirty bitmaps, Vladimir Sementsov-Ogievskiy, 2018/02/07
- [Qemu-block] ping Re: [PATCH v10 00/12] Dirty bitmaps postcopy migration, Vladimir Sementsov-Ogievskiy, 2018/02/15
- Re: [Qemu-block] [PATCH v10 00/12] Dirty bitmaps postcopy migration, Vladimir Sementsov-Ogievskiy, 2018/02/16