[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 4/6] block/dirty-bitmaps: prohibit enable/dis
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [Qemu-devel] [PATCH v4 4/6] block/dirty-bitmaps: prohibit enable/disable on locked/frozen bitmaps |
Date: |
Wed, 3 Oct 2018 13:05:27 +0000 |
03.10.2018 02:02, John Snow wrote:
> 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>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> ---
> blockdev.c | 26 ++++++++++++++++++++------
> 1 file changed, 20 insertions(+), 6 deletions(-)
>
> diff --git a/blockdev.c b/blockdev.c
> index cccd36b5e7..d0febfca79 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_locked(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_locked(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);
> }
> @@ -2933,10 +2947,10 @@ 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_locked(bitmap)) {
> error_setg(errp,
> - "Bitmap '%s' is currently frozen and cannot be enabled",
> - name);
> + "Bitmap '%s' is currently in use by another operation"
> + " and cannot be enabled", name);
> return;
> }
>
> @@ -2954,10 +2968,10 @@ 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_locked(bitmap)) {
> error_setg(errp,
> - "Bitmap '%s' is currently frozen and cannot be disabled",
> - name);
> + "Bitmap '%s' is currently in use by another operation"
> + " and cannot be disabled", name);
> return;
> }
>
--
Best regards,
Vladimir
- [Qemu-devel] [PATCH v4 0/6] dirty-bitmaps: fix QMP command permissions, John Snow, 2018/10/02
- [Qemu-devel] [PATCH v4 6/6] nbd: forbid use of frozen bitmaps, John Snow, 2018/10/02
- Re: [Qemu-devel] [PATCH v4 0/6] dirty-bitmaps: fix QMP command permissions, John Snow, 2018/10/02
- [Qemu-devel] [PATCH v4 4/6] block/dirty-bitmaps: prohibit enable/disable on locked/frozen bitmaps, John Snow, 2018/10/02
- [Qemu-devel] [PATCH v4 2/6] block/dirty-bitmaps: fix merge permissions, John Snow, 2018/10/02
- [Qemu-devel] [PATCH v4 3/6] block/dirty-bitmaps: allow clear on disabled bitmaps, John Snow, 2018/10/02
- [Qemu-devel] [PATCH v4 5/6] block/backup: prohibit backup from using in use bitmaps, John Snow, 2018/10/02