From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34859) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YOEHB-0002M8-M2 for qemu-devel@nongnu.org; Wed, 18 Feb 2015 18:43:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YOEHA-0003nE-Ei for qemu-devel@nongnu.org; Wed, 18 Feb 2015 18:43:29 -0500 Received: from mx1.redhat.com ([209.132.183.28]:40756) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YOEHA-0003nA-8E for qemu-devel@nongnu.org; Wed, 18 Feb 2015 18:43:28 -0500 Message-ID: <54E5239C.8090109@redhat.com> Date: Wed, 18 Feb 2015 18:43:24 -0500 From: John Snow MIME-Version: 1.0 References: <1424268014-13293-1-git-send-email-vsementsov@parallels.com> <1424268014-13293-4-git-send-email-vsementsov@parallels.com> In-Reply-To: <1424268014-13293-4-git-send-email-vsementsov@parallels.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH RFC v3 03/14] block: BdrvDirtyBitmap serialization interface List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Vladimir Sementsov-Ogievskiy , qemu-devel@nongnu.org Cc: kwolf@redhat.com, peter.maydell@linaro.org, quintela@redhat.com, dgilbert@redhat.com, stefanha@redhat.com, pbonzini@redhat.com, amit.shah@redhat.com, den@openvz.org On 02/18/2015 09:00 AM, Vladimir Sementsov-Ogievskiy wrote: > Several functions to provide necessary access to BdrvDirtyBitmap for > block-migration.c > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block.c | 36 ++++++++++++++++++++++++++++++++++++ > include/block/block.h | 13 +++++++++++++ > 2 files changed, 49 insertions(+) > > diff --git a/block.c b/block.c > index e4547d7..e95a5ae 100644 > --- a/block.c > +++ b/block.c > @@ -5674,6 +5674,42 @@ void bdrv_clear_dirty_bitmap(BdrvDirtyBitmap *bitmap) > hbitmap_reset(bitmap->bitmap, 0, bitmap->size); > } > > +const char *bdrv_dirty_bitmap_name(const BdrvDirtyBitmap *bitmap) > +{ > + return bitmap->name; > +} > + > +uint64_t bdrv_dirty_bitmap_data_size(const BdrvDirtyBitmap *bitmap, > + uint64_t count) > +{ > + return hbitmap_data_size(bitmap->bitmap, count); > +} > + > +void bdrv_dirty_bitmap_serialize_part(const BdrvDirtyBitmap *bitmap, > + uint8_t *buf, uint64_t start, > + uint64_t count) > +{ > + hbitmap_serialize_part(bitmap->bitmap, buf, start, count); > +} > + > +void bdrv_dirty_bitmap_deserialize_part(BdrvDirtyBitmap *bitmap, > + uint8_t *buf, uint64_t start, > + uint64_t count) > +{ > + hbitmap_deserialize_part(bitmap->bitmap, buf, start, count); > +} > + > +void bdrv_dirty_bitmap_deserialize_zeroes(BdrvDirtyBitmap *bitmap, > + uint64_t start, uint64_t count) > +{ > + hbitmap_deserialize_zeroes(bitmap->bitmap, start, count); > +} > + > +void bdrv_dirty_bitmap_deserialize_finish(BdrvDirtyBitmap *bitmap) > +{ > + hbitmap_deserialize_finish(bitmap->bitmap); > +} > + > static void bdrv_set_dirty(BlockDriverState *bs, int64_t cur_sector, > int nr_sectors) > { > diff --git a/include/block/block.h b/include/block/block.h > index c9d96a6..c6a928d 100644 > --- a/include/block/block.h > +++ b/include/block/block.h > @@ -474,6 +474,19 @@ void bdrv_dirty_iter_init(BdrvDirtyBitmap *bitmap, struct HBitmapIter *hbi); > void bdrv_set_dirty_iter(struct HBitmapIter *hbi, int64_t offset); > int64_t bdrv_get_dirty_count(BdrvDirtyBitmap *bitmap); > > +const char *bdrv_dirty_bitmap_name(const BdrvDirtyBitmap *bitmap); > +uint64_t bdrv_dirty_bitmap_data_size(const BdrvDirtyBitmap *bitmap, > + uint64_t count); > +void bdrv_dirty_bitmap_serialize_part(const BdrvDirtyBitmap *bitmap, > + uint8_t *buf, uint64_t start, > + uint64_t count); > +void bdrv_dirty_bitmap_deserialize_part(BdrvDirtyBitmap *bitmap, > + uint8_t *buf, uint64_t start, > + uint64_t count); > +void bdrv_dirty_bitmap_deserialize_zeroes(BdrvDirtyBitmap *bitmap, > + uint64_t start, uint64_t count); > +void bdrv_dirty_bitmap_deserialize_finish(BdrvDirtyBitmap *bitmap); > + > void bdrv_enable_copy_on_read(BlockDriverState *bs); > void bdrv_disable_copy_on_read(BlockDriverState *bs); > > Reviewed-by: John Snow