[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v2 1/4] drity-bitmap: switch assert
From: |
John Snow |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v2 1/4] drity-bitmap: switch assert-fails to errors in bdrv_merge_dirty_bitmap |
Date: |
Mon, 10 Sep 2018 17:02:14 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
s/drity-bitmap/dirty-bitmap/g
On 07/06/2018 07:36 AM, Vladimir Sementsov-Ogievskiy wrote:
> Move checks from qmp_x_block_dirty_bitmap_merge() to
> bdrv_merge_dirty_bitmap(), to share them with dirty bitmap merge
> transaction action in future commit.
>
> Note: for now, only qmp_x_block_dirty_bitmap_merge() calls
> bdrv_merge_dirty_bitmap().
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> ---
> block/dirty-bitmap.c | 15 +++++++++++++--
> blockdev.c | 10 ----------
> 2 files changed, 13 insertions(+), 12 deletions(-)
>
> diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c
> index c9b8a6fd52..6c8761e027 100644
> --- a/block/dirty-bitmap.c
> +++ b/block/dirty-bitmap.c
> @@ -798,12 +798,23 @@ void bdrv_merge_dirty_bitmap(BdrvDirtyBitmap *dest,
> const BdrvDirtyBitmap *src,
>
> qemu_mutex_lock(dest->mutex);
>
> - assert(bdrv_dirty_bitmap_enabled(dest));
> - assert(!bdrv_dirty_bitmap_readonly(dest));
> + if (bdrv_dirty_bitmap_frozen(dest)) {
> + error_setg(errp, "Bitmap '%s' is frozen and cannot be modified",
> + dest->name);
> + goto out;
> + }
> +
> + if (bdrv_dirty_bitmap_readonly(dest)) {
> + error_setg(errp, "Bitmap '%s' is readonly and cannot be modified",
> + dest->name);
> + goto out;
> + }
>
> if (!hbitmap_merge(dest->bitmap, src->bitmap)) {
> error_setg(errp, "Bitmaps are incompatible and can't be merged");
> + goto out;
This is unusual right before the out: segment, but harmless.
> }
>
> +out:
> qemu_mutex_unlock(dest->mutex);
> }
> diff --git a/blockdev.c b/blockdev.c
> index 72f5347df5..902338e815 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -2955,16 +2955,6 @@ void qmp_x_block_dirty_bitmap_merge(const char *node,
> const char *dst_name,
> return;
> }
>
> - if (bdrv_dirty_bitmap_frozen(dst)) {
> - error_setg(errp, "Bitmap '%s' is frozen and cannot be modified",
> - dst_name);
> - return;
> - } else if (bdrv_dirty_bitmap_readonly(dst)) {
> - error_setg(errp, "Bitmap '%s' is readonly and cannot be modified",
> - dst_name);
> - return;
> - }
> -
> src = bdrv_find_dirty_bitmap(bs, src_name);
> if (!src) {
> error_setg(errp, "Dirty bitmap '%s' not found", src_name);
>
When fixing the commit title, we can add:
Reviewed-by: John Snow <address@hidden>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 1/4] drity-bitmap: switch assert-fails to errors in bdrv_merge_dirty_bitmap,
John Snow <=