[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/10] block: fix backup in vmdk format image
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 09/10] block: fix backup in vmdk format image |
Date: |
Fri, 27 May 2016 11:01:36 -0700 |
User-agent: |
Mutt/1.6.0 (2016-04-01) |
On Sat, May 14, 2016 at 03:45:57PM +0300, Denis V. Lunev wrote:
> From: Pavel Butsykin <address@hidden>
>
> The vmdk format has the extents and bs->file can be equal to the first
> extension. Before start of the backup we do detach the old context on the
> target drive at the bdrv_attach_aio_context. For the vmdk drive this means
> a double detach of the same block driver state, because the detach occurs
> for s->extents[0].file and bs->file.
>
> To fix we just skip the detach if s->extents[i].file and bs->file are the
> same. This approach is already used in the vmdk_free_extents() and the
> vmdk_get_allocated_file_size(), so it won't be some innovation :)
>
> Signed-off-by: Pavel Butsykin <address@hidden>
> Signed-off-by: Denis V. Lunev <address@hidden>
> CC: Jeff Cody <address@hidden>
> CC: Markus Armbruster <address@hidden>
> CC: Eric Blake <address@hidden>
> CC: John Snow <address@hidden>
> CC: Stefan Hajnoczi <address@hidden>
> CC: Kevin Wolf <address@hidden>
> ---
> block/vmdk.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/block/vmdk.c b/block/vmdk.c
> index 9530b30..0550924 100644
> --- a/block/vmdk.c
> +++ b/block/vmdk.c
> @@ -2305,7 +2305,10 @@ static void vmdk_detach_aio_context(BlockDriverState
> *bs)
> int i;
>
> for (i = 0; i < s->num_extents; i++) {
> - bdrv_detach_aio_context(s->extents[i].file->bs);
> + BdrvChild *file = s->extents[i].file;
> + if (file != bs->file) {
> + bdrv_detach_aio_context(file->bs);
> + }
> }
> }
I think this fix is no longer necessary. Max eliminated
vmdk_detach_aio_context() here:
[PULL 24/31] block: Propagate AioContext change to all children
signature.asc
Description: PGP signature
- [Qemu-devel] [PATCH 05/10] block: remove BlockDriver.bdrv_write_compressed, (continued)
- [Qemu-devel] [PATCH 05/10] block: remove BlockDriver.bdrv_write_compressed, Denis V. Lunev, 2016/05/14
- [Qemu-devel] [PATCH 07/10] blockdev-backup: added support for data compression, Denis V. Lunev, 2016/05/14
- [Qemu-devel] [PATCH 06/10] drive-backup: added support for data compression, Denis V. Lunev, 2016/05/14
- [Qemu-devel] [PATCH 09/10] block: fix backup in vmdk format image, Denis V. Lunev, 2016/05/14
- Re: [Qemu-devel] [PATCH 09/10] block: fix backup in vmdk format image,
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH 08/10] qemu-iotests: test backup compression in 055, Denis V. Lunev, 2016/05/14