ae23c9
From 5944f5aa58219777441158766624e571b53de954 Mon Sep 17 00:00:00 2001
ae23c9
From: John Snow <jsnow@redhat.com>
ae23c9
Date: Tue, 20 Nov 2018 18:18:05 +0000
ae23c9
Subject: [PATCH 11/35] block: remove bdrv_dirty_bitmap_make_anon
ae23c9
ae23c9
RH-Author: John Snow <jsnow@redhat.com>
ae23c9
Message-id: <20181120181828.15132-2-jsnow@redhat.com>
ae23c9
Patchwork-id: 83052
ae23c9
O-Subject: [RHEL8/rhel qemu-kvm PATCH 01/24] block: remove bdrv_dirty_bitmap_make_anon
ae23c9
Bugzilla: 1518989
ae23c9
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
ae23c9
RH-Acked-by: Max Reitz <mreitz@redhat.com>
ae23c9
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
ae23c9
ae23c9
From: Paolo Bonzini <pbonzini@redhat.com>
ae23c9
ae23c9
All this function is doing will be repeated by
ae23c9
bdrv_do_release_matching_dirty_bitmap_locked, except
ae23c9
resetting bm->persistent.  But even that does not matter
ae23c9
because the bitmap will be freed.
ae23c9
ae23c9
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
ae23c9
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
ae23c9
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
ae23c9
Message-id: 20180323164254.26487-1-pbonzini@redhat.com
ae23c9
Signed-off-by: John Snow <jsnow@redhat.com>
ae23c9
(cherry picked from commit ab41fc4853cc0cf01ed4903ffe7c36e3768b538f)
ae23c9
Signed-off-by: John Snow <jsnow@redhat.com>
ae23c9
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
ae23c9
---
ae23c9
 block/dirty-bitmap.c         | 9 ---------
ae23c9
 blockdev.c                   | 1 -
ae23c9
 include/block/dirty-bitmap.h | 1 -
ae23c9
 3 files changed, 11 deletions(-)
ae23c9
ae23c9
diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c
ae23c9
index f580c1a..634f143 100644
ae23c9
--- a/block/dirty-bitmap.c
ae23c9
+++ b/block/dirty-bitmap.c
ae23c9
@@ -98,15 +98,6 @@ BdrvDirtyBitmap *bdrv_find_dirty_bitmap(BlockDriverState *bs, const char *name)
ae23c9
 }
ae23c9
 
ae23c9
 /* Called with BQL taken.  */
ae23c9
-void bdrv_dirty_bitmap_make_anon(BdrvDirtyBitmap *bitmap)
ae23c9
-{
ae23c9
-    assert(!bdrv_dirty_bitmap_frozen(bitmap));
ae23c9
-    g_free(bitmap->name);
ae23c9
-    bitmap->name = NULL;
ae23c9
-    bitmap->persistent = false;
ae23c9
-}
ae23c9
-
ae23c9
-/* Called with BQL taken.  */
ae23c9
 BdrvDirtyBitmap *bdrv_create_dirty_bitmap(BlockDriverState *bs,
ae23c9
                                           uint32_t granularity,
ae23c9
                                           const char *name,
ae23c9
diff --git a/blockdev.c b/blockdev.c
ae23c9
index 70af034..69610e7 100644
ae23c9
--- a/blockdev.c
ae23c9
+++ b/blockdev.c
ae23c9
@@ -2977,7 +2977,6 @@ void qmp_block_dirty_bitmap_remove(const char *node, const char *name,
ae23c9
         }
ae23c9
     }
ae23c9
 
ae23c9
-    bdrv_dirty_bitmap_make_anon(bitmap);
ae23c9
     bdrv_release_dirty_bitmap(bs, bitmap);
ae23c9
 }
ae23c9
 
ae23c9
diff --git a/include/block/dirty-bitmap.h b/include/block/dirty-bitmap.h
ae23c9
index af9ba3c..bf68dd7 100644
ae23c9
--- a/include/block/dirty-bitmap.h
ae23c9
+++ b/include/block/dirty-bitmap.h
ae23c9
@@ -24,7 +24,6 @@ BdrvDirtyBitmap *bdrv_reclaim_dirty_bitmap(BlockDriverState *bs,
ae23c9
 void bdrv_dirty_bitmap_enable_successor(BdrvDirtyBitmap *bitmap);
ae23c9
 BdrvDirtyBitmap *bdrv_find_dirty_bitmap(BlockDriverState *bs,
ae23c9
                                         const char *name);
ae23c9
-void bdrv_dirty_bitmap_make_anon(BdrvDirtyBitmap *bitmap);
ae23c9
 void bdrv_release_dirty_bitmap(BlockDriverState *bs, BdrvDirtyBitmap *bitmap);
ae23c9
 void bdrv_release_named_dirty_bitmaps(BlockDriverState *bs);
ae23c9
 void bdrv_release_persistent_dirty_bitmaps(BlockDriverState *bs);
ae23c9
-- 
ae23c9
1.8.3.1
ae23c9