[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [RFC PATCH v2 5/6] bitmaps: prohibit enable/disable on lock
From: |
John Snow |
Subject: |
[Qemu-devel] [RFC PATCH v2 5/6] bitmaps: prohibit enable/disable on locked/frozen bitmaps |
Date: |
Fri, 21 Sep 2018 18:28:46 -0400 |
We're not being consistent about this. If it's in use by an operation,
the user should not be able to change the behavior of that bitmap.
Signed-off-by: John Snow <address@hidden>
---
blockdev.c | 22 ++++++++++++++++++----
1 file changed, 18 insertions(+), 4 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index 070180898f..651c50d1fa 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -2058,6 +2058,13 @@ static void
block_dirty_bitmap_enable_prepare(BlkActionState *common,
return;
}
+ if (!bdrv_dirty_bitmap_user_modifiable(state->bitmap)) {
+ error_setg(errp,
+ "Bitmap '%s' is currently in-use by another operation and
cannot be enabled",
+ action->name);
+ return;
+ }
+
state->was_enabled = bdrv_dirty_bitmap_enabled(state->bitmap);
bdrv_enable_dirty_bitmap(state->bitmap);
}
@@ -2092,6 +2099,13 @@ static void
block_dirty_bitmap_disable_prepare(BlkActionState *common,
return;
}
+ if (!bdrv_dirty_bitmap_user_modifiable(state->bitmap)) {
+ error_setg(errp,
+ "Bitmap '%s' is currently in-use by another operation and
cannot be disabled",
+ action->name);
+ return;
+ }
+
state->was_enabled = bdrv_dirty_bitmap_enabled(state->bitmap);
bdrv_disable_dirty_bitmap(state->bitmap);
}
@@ -2892,9 +2906,9 @@ void qmp_x_block_dirty_bitmap_enable(const char *node,
const char *name,
return;
}
- if (bdrv_dirty_bitmap_frozen(bitmap)) {
+ if (!bdrv_dirty_bitmap_user_modifiable(bitmap)) {
error_setg(errp,
- "Bitmap '%s' is currently frozen and cannot be enabled",
+ "Bitmap '%s' is currently in-use by another operation and
cannot be enabled",
name);
return;
}
@@ -2913,9 +2927,9 @@ void qmp_x_block_dirty_bitmap_disable(const char *node,
const char *name,
return;
}
- if (bdrv_dirty_bitmap_frozen(bitmap)) {
+ if (!bdrv_dirty_bitmap_user_modifiable(bitmap)) {
error_setg(errp,
- "Bitmap '%s' is currently frozen and cannot be disabled",
+ "Bitmap '%s' is currently in-use by another operation and
cannot be disabled",
name);
return;
}
--
2.14.4
- [Qemu-devel] [RFC PATCH v2 0/6] dirty-bitmaps: fix QMP command permissions, John Snow, 2018/09/21
- [Qemu-devel] [RFC PATCH v2 1/6] dirty-bitmaps: allow merging to disabled bitmaps, John Snow, 2018/09/21
- [Qemu-devel] [RFC PATCH v2 2/6] bitmaps: allow clear on disabled bitmaps, John Snow, 2018/09/21
- [Qemu-devel] [RFC PATCH v2 3/6] bitmaps: prohibit merge on locked bitmaps, John Snow, 2018/09/21
- [Qemu-devel] [RFC PATCH v2 4/6] bitmaps: add user_modifiable status checker, John Snow, 2018/09/21
- [Qemu-devel] [RFC PATCH v2 5/6] bitmaps: prohibit enable/disable on locked/frozen bitmaps,
John Snow <=
- [Qemu-devel] [RFC PATCH v2 6/6] block/backup: prohibit backup from using in-use bitmaps, John Snow, 2018/09/21