[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/12] block: add meta bitmaps
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH 04/12] block: add meta bitmaps |
Date: |
Thu, 28 May 2015 18:13:44 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
On 05/13/2015 11:29 AM, Vladimir Sementsov-Ogievskiy wrote:
> Meta bitmap is a 'dirty bitmap' for the BdrvDirtyBitmap. It tracks
> changes (set/unset) of this BdrvDirtyBitmap. It is needed for live
> migration of block dirty bitmaps.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> ---
> block.c | 44 ++++++++++++++++++++++++++++++++++++++++++++
> include/block/block.h | 7 +++++++
> 2 files changed, 51 insertions(+)
>
> diff --git a/block.c b/block.c
> index 8376e8d..a167eb6 100644
> --- a/block.c
> +++ b/block.c
> @@ -58,9 +58,15 @@
> * (3) successor is set: frozen mode.
> * A frozen bitmap cannot be renamed, deleted, anonymized, cleared, set,
> * or enabled. A frozen bitmap can only abdicate() or reclaim().
> + *
> + * Meta bitmap:
> + * Meta bitmap is a 'dirty bitmap' for the BdrvDirtyBitmap. It tracks changes
> + * (set/unset) of this BdrvDirtyBitmap. It is needed for live migration of
> + * block dirty bitmaps.
> */
> struct BdrvDirtyBitmap {
> HBitmap *bitmap; /* Dirty sector bitmap implementation */
> + HBitmap *meta_bitmap; /* Meta bitmap */
> BdrvDirtyBitmap *successor; /* Anonymous child; implies frozen status */
> char *name; /* Optional non-empty unique ID */
> int64_t size; /* Size of the bitmap (Number of sectors) */
> @@ -3062,6 +3068,35 @@ void bdrv_dirty_bitmap_make_anon(BdrvDirtyBitmap
> *bitmap)
> bitmap->name = NULL;
> }
>
> +HBitmap *bdrv_create_meta_bitmap(BdrvDirtyBitmap *bitmap,
> + uint64_t chunk_size)
> +{
> + uint64_t sector_granularity;
> +
> + assert((chunk_size & (chunk_size - 1)) == 0);
> +
> + /* one chunk is corresponding to one bit of the meta bitmap, and each bit
> + * of the chunk is corresponding to
> 'bdrv_dirty_bitmap_granularity(bitmap)'
> + * bytes of the node */
> + sector_granularity =
> + (chunk_size * 8 * bdrv_dirty_bitmap_granularity(bitmap))
> + >> BDRV_SECTOR_BITS;
> + assert(sector_granularity);
> +
> + bitmap->meta_bitmap =
> + hbitmap_alloc(bitmap->size, ffsll(sector_granularity) - 1);
> +
> + return bitmap->meta_bitmap;
> +}
> +
> +void bdrv_release_meta_bitmap(BdrvDirtyBitmap *bitmap)
> +{
> + if (bitmap->meta_bitmap) {
> + hbitmap_free(bitmap->meta_bitmap);
> + bitmap->meta_bitmap = NULL;
> + }
> +}
> +
> BdrvDirtyBitmap *bdrv_create_dirty_bitmap(BlockDriverState *bs,
> uint32_t granularity,
> const char *name,
> @@ -3212,6 +3247,9 @@ void bdrv_release_dirty_bitmap(BlockDriverState *bs,
> BdrvDirtyBitmap *bitmap)
> assert(!bdrv_dirty_bitmap_frozen(bm));
> QLIST_REMOVE(bitmap, list);
> hbitmap_free(bitmap->bitmap);
> + if (bitmap->meta_bitmap) {
> + hbitmap_free(bitmap->meta_bitmap);
> + }
> g_free(bitmap->name);
> g_free(bitmap);
> return;
> @@ -3297,6 +3335,9 @@ void bdrv_set_dirty_bitmap(BdrvDirtyBitmap *bitmap,
> {
> assert(bdrv_dirty_bitmap_enabled(bitmap));
> hbitmap_set(bitmap->bitmap, cur_sector, nr_sectors);
> + if (bitmap->meta_bitmap) {
> + hbitmap_set(bitmap->meta_bitmap, cur_sector, nr_sectors);
> + }
> }
>
> void bdrv_reset_dirty_bitmap(BdrvDirtyBitmap *bitmap,
> @@ -3304,6 +3345,9 @@ void bdrv_reset_dirty_bitmap(BdrvDirtyBitmap *bitmap,
> {
> assert(bdrv_dirty_bitmap_enabled(bitmap));
> hbitmap_reset(bitmap->bitmap, cur_sector, nr_sectors);
> + if (bitmap->meta_bitmap) {
> + hbitmap_set(bitmap->meta_bitmap, cur_sector, nr_sectors);
> + }
> }
>
> void bdrv_clear_dirty_bitmap(BdrvDirtyBitmap *bitmap)
> diff --git a/include/block/block.h b/include/block/block.h
> index 1c16906..2d91161 100644
> --- a/include/block/block.h
> +++ b/include/block/block.h
> @@ -4,6 +4,7 @@
> #include "block/aio.h"
> #include "qemu-common.h"
> #include "qemu/option.h"
> +#include "qemu/hbitmap.h"
> #include "block/coroutine.h"
> #include "block/accounting.h"
> #include "qapi/qmp/qobject.h"
> @@ -497,6 +498,12 @@ void
> bdrv_dirty_bitmap_deserialize_zeroes(BdrvDirtyBitmap *bitmap,
> uint64_t start, uint64_t count);
> void bdrv_dirty_bitmap_deserialize_finish(BdrvDirtyBitmap *bitmap);
>
> +/* chunk size here is number of bytes of the @bitmap data per one bit of the
> + * meta bitmap being created */
> +HBitmap *bdrv_create_meta_bitmap(BdrvDirtyBitmap *bitmap,
> + uint64_t granularity);
> +void bdrv_release_meta_bitmap(BdrvDirtyBitmap *bitmap);
> +
> void bdrv_enable_copy_on_read(BlockDriverState *bs);
> void bdrv_disable_copy_on_read(BlockDriverState *bs);
>
>
Reviewed-by: John Snow <address@hidden>
- [Qemu-devel] [PATCH 03/12] block: tiny refactoring: minimize hbitmap_(set/reset) usage, (continued)
- [Qemu-devel] [PATCH 03/12] block: tiny refactoring: minimize hbitmap_(set/reset) usage, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 09/12] iotests: add add_incoming_migration to VM class, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 08/12] iotests: maintain several vms in test, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 05/12] block: add bdrv_next_dirty_bitmap(), Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 06/12] qapi: add dirty-bitmaps migration capability, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 12/12] migration/qemu-file: make functions qemu_(get/put)_string public, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 10/12] qapi: add md5 checksum of last dirty bitmap level to query-block, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 02/12] block: BdrvDirtyBitmap serialization interface, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 11/12] iotests: add dirty bitmap migration test, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 04/12] block: add meta bitmaps, Vladimir Sementsov-Ogievskiy, 2015/05/13
- Re: [Qemu-devel] [PATCH 04/12] block: add meta bitmaps,
John Snow <=
- [Qemu-devel] [PATCH 01/12] hbitmap: serialization, Vladimir Sementsov-Ogievskiy, 2015/05/13
- [Qemu-devel] [PATCH 07/12] migration: add migration/block-dirty-bitmap.c, Vladimir Sementsov-Ogievskiy, 2015/05/13
- Re: [Qemu-devel] [PATCH v5 00/12] Dirty bitmaps migration, Vladimir Sementsov-Ogievskiy, 2015/05/21
- Re: [Qemu-devel] [PATCH v5 00/12] Dirty bitmaps migration, Denis V. Lunev, 2015/05/21
- Re: [Qemu-devel] [PATCH v5 00/12] Dirty bitmaps migration, John Snow, 2015/05/21
- Re: [Qemu-devel] [PATCH v5 00/12] Dirty bitmaps migration, Denis V. Lunev, 2015/05/26
- Re: [Qemu-devel] [PATCH v5 00/12] Dirty bitmaps migration, Denis V. Lunev, 2015/05/26
- Re: [Qemu-devel] [PATCH v5 00/12] Dirty bitmaps migration, John Snow, 2015/05/28
- Re: [Qemu-devel] [PATCH v5 00/12] Dirty bitmaps migration, Denis V. Lunev, 2015/05/28