[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/5] block: bdrv_insert_node(): don't use bdrv_open()
From: |
Kevin Wolf |
Subject: |
Re: [PATCH 4/5] block: bdrv_insert_node(): don't use bdrv_open() |
Date: |
Tue, 21 Sep 2021 16:33:45 +0200 |
Am 20.09.2021 um 13:55 hat Vladimir Sementsov-Ogievskiy geschrieben:
> Use bdrv_new_open_driver_opts() instead of complicated bdrv_open().
>
> Among other extra things bdrv_open() also check for white-listed
> formats, which we don't want for internal node creation: currently
> backup doesn't work when copy-before-write filter is not white-listed.
> As well block-stream doesn't work when copy-on-read is not
> white-listed.
>
> Fixes: 751cec7a261adaf1145dc7adf6de7c9c084e5a0b
> Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2004812
> Reported-by: Yanan Fu
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
> ---
> block.c | 32 +++++++++++++++++++++++++++-----
> 1 file changed, 27 insertions(+), 5 deletions(-)
>
> diff --git a/block.c b/block.c
> index 3a90407b83..a174801785 100644
> --- a/block.c
> +++ b/block.c
> @@ -5134,11 +5134,29 @@ BlockDriverState *bdrv_insert_node(BlockDriverState
> *bs, QDict *options,
> ERRP_GUARD();
> int ret;
> BlockDriverState *new_node_bs;
gcc tells me that this needs to be initialised now because of the
bdrv_unref() aftert the fail: label.
Fixed this up to be new_node_bs = NULL and applied the series to the
block branch, thanks.
Kevin
- [PATCH 0/5] Fix not white-listed copy-before-write, Vladimir Sementsov-Ogievskiy, 2021/09/20
- [PATCH 1/5] block: implement bdrv_new_open_driver_opts(), Vladimir Sementsov-Ogievskiy, 2021/09/20
- [PATCH 2/5] block: bdrv_insert_node(): fix and improve error handling, Vladimir Sementsov-Ogievskiy, 2021/09/20
- [PATCH 3/5] block: bdrv_insert_node(): doc and style, Vladimir Sementsov-Ogievskiy, 2021/09/20
- [PATCH 5/5] iotests/image-fleecing: declare requirement of copy-before-write, Vladimir Sementsov-Ogievskiy, 2021/09/20
- [PATCH 4/5] block: bdrv_insert_node(): don't use bdrv_open(), Vladimir Sementsov-Ogievskiy, 2021/09/20
- Re: [PATCH 4/5] block: bdrv_insert_node(): don't use bdrv_open(),
Kevin Wolf <=