[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] block: Close backing file early in bdrv_img_
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v3] block: Close backing file early in bdrv_img_create |
Date: |
Tue, 3 Dec 2013 16:21:28 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 03.12.2013 um 14:57 hat Max Reitz geschrieben:
> 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).
>
> In addition, the BlockDriverState in bdrv_img_create() is used for the
> backing file only; it should therefore be made local to the respective
> block.
>
> Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Kevin Wolf <address@hidden>