[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] block: Close backing file early in bdrv_img_
From: |
Wenchao Xia |
Subject: |
Re: [Qemu-devel] [PATCH v2] block: Close backing file early in bdrv_img_create |
Date: |
Mon, 02 Dec 2013 11:37:36 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.1.1 |
δΊ 2013/11/30 4:41, Max Reitz ει:
> Leaving the backing file open although it is not needed anymore can
> cause problems if it is opened through a block driver which allows
> exclusive access only and if the create function of the block driver
> used for the top image (the one being created) tries to close and reopen
> the image file (which will include opening the backing file a second
> time).
>
> In particular, this will happen with a backing file opened through
> qemu-nbd and using qcow2 as the top image file format (which reopens the
> image to flush it to disk).
>
> Signed-off-by: Max Reitz <address@hidden>
> ---
> v2:
> - Minimizing the changes prevents introducing a leak of the
> BlockDriverState in case of an error in bdrv_open() (thanks, Kevin).
> ---
> block.c | 3 +++
> 1 file changed, 3 insertions(+)
>
Minor comments:
I think your v1 have better orgnize of code, since it tips reader
that bs is a variable used only in backing file code. Why not improve
it by just adding one line in v1:
line 4587:
ret = bdrv_open(bs, backing_file->value.s, NULL, back_flags,
backing_drv, &local_err);
if (ret < 0) {
error_setg_errno(errp, -ret, "Could not open '%s': %s",
backing_file->value.s,
error_get_pretty(local_err));
error_free(local_err);
local_err = NULL;
bdrv_unref(bs);
goto out;
}
It is not a big problem, I am OK if you stick to this version.
- Re: [Qemu-devel] [PATCH v2] block: Close backing file early in bdrv_img_create,
Wenchao Xia <=