[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 05/10] qcow2/bitmap: reject IN_USE bitmaps on rw
From: |
John Snow |
Subject: |
[Qemu-devel] [PATCH v2 05/10] qcow2/bitmap: reject IN_USE bitmaps on rw reload |
Date: |
Tue, 12 Jun 2018 22:06:08 -0400 |
Presently, an image with IN_USE bitmaps cannot be loaded as RO.
In preparation for changing that, IN_USE bitmaps ought to create an
error on reload instead of being skipped.
While we're here, update the function to work with two new facts:
- All bm_list entries will have a BdrvDirtyBitmap (because we load and
disable bitmaps when the autoload flag isn't set), and
- The dirty_bitmap will be cached, so we don't have to look it up.
Signed-off-by: John Snow <address@hidden>
---
block/qcow2-bitmap.c | 31 +++++++++++++++----------------
1 file changed, 15 insertions(+), 16 deletions(-)
diff --git a/block/qcow2-bitmap.c b/block/qcow2-bitmap.c
index d94b6bd5cf..859819639b 100644
--- a/block/qcow2-bitmap.c
+++ b/block/qcow2-bitmap.c
@@ -1046,23 +1046,22 @@ int qcow2_reopen_bitmaps_rw_hint(BlockDriverState *bs,
bool *header_updated,
}
QSIMPLEQ_FOREACH(bm, bm_list, entry) {
- if (!(bm->flags & BME_FLAG_IN_USE)) {
- BdrvDirtyBitmap *bitmap = bdrv_find_dirty_bitmap(bs, bm->name);
- if (bitmap == NULL) {
- continue;
- }
-
- if (!bdrv_dirty_bitmap_readonly(bitmap)) {
- error_setg(errp, "Bitmap %s is not readonly but not marked"
- "'IN_USE' in the image. Something went wrong,"
- "all the bitmaps may be corrupted", bm->name);
- ret = -EINVAL;
- goto out;
- }
-
- bm->flags |= BME_FLAG_IN_USE;
- ro_dirty_bitmaps = g_slist_append(ro_dirty_bitmaps, bitmap);
+ if (bm->flags & BME_FLAG_IN_USE) {
+ error_setg(errp, "Bitmap '%s' is in use; can't reopen RW",
+ bm->name);
+ ret = -EINVAL;
+ goto out;
+ } else if (!bdrv_dirty_bitmap_readonly(bm->dirty_bitmap)) {
+ error_setg(errp, "Bitmap %s is neither readonly nor 'IN_USE' in "
+ "the image. Something went wrong, all the bitmaps may "
+ "be corrupted. Please report this to the developers at "
+ " address@hidden", bm->name);
+ ret = -EINVAL;
+ goto out;
}
+
+ bm->flags |= BME_FLAG_IN_USE;
+ ro_dirty_bitmaps = g_slist_append(ro_dirty_bitmaps, bm->dirty_bitmap);
}
if (ro_dirty_bitmaps != NULL) {
--
2.14.3
- [Qemu-devel] [PATCH v2 00/10] qemu-img: add bitmap info output, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 04/10] qcow2/bitmap: cache loaded bitmaps, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 08/10] qcow2/bitmap: track extra_data_size, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 07/10] qcow2/bitmap: track bitmap type, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 05/10] qcow2/bitmap: reject IN_USE bitmaps on rw reload,
John Snow <=
- [Qemu-devel] [PATCH v2 02/10] qcow2/bitmap: avoid adjusting bm->flags for RO bitmaps, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 09/10] qapi: add bitmap info, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 01/10] qcow2/bitmap: remove redundant arguments from bitmap_list_load, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 06/10] qcow2/bitmap: load IN_USE bitmaps if disk is RO, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 10/10] qcow2/bitmap: add basic bitmaps info, John Snow, 2018/06/12
- [Qemu-devel] [PATCH v2 03/10] qcow2/bitmap: cache bm_list, John Snow, 2018/06/12