[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 1/3] block: resize backing file image during
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v4 1/3] block: resize backing file image during offline commit, if necessary |
Date: |
Fri, 24 Jan 2014 12:42:25 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 23.01.2014 um 23:45 hat Jeff Cody geschrieben:
> Currently, if an image file is logically larger than its backing file,
> committing it via 'qemu-img commit' will fail.
>
> For instance, if we have a base image with a virtual size 10G, and a
> snapshot image of size 20G, then committing the snapshot offline with
> 'qemu-img commit' will likely fail.
>
> This will automatically attempt to resize the base image, if the
> snapshot image to be committed is larger.
>
> Signed-off-by: Jeff Cody <address@hidden>
> Reviewed-by: Fam Zheng <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> Reviewed-by: Benoit Canet <address@hidden>
> ---
> block.c | 23 ++++++++++++++++++++---
> 1 file changed, 20 insertions(+), 3 deletions(-)
>
> diff --git a/block.c b/block.c
> index 64e7d22..dea7591 100644
> --- a/block.c
> +++ b/block.c
> @@ -1876,10 +1876,10 @@ int bdrv_check(BlockDriverState *bs, BdrvCheckResult
> *res, BdrvCheckMode fix)
> int bdrv_commit(BlockDriverState *bs)
> {
> BlockDriver *drv = bs->drv;
> - int64_t sector, total_sectors;
> + int64_t sector, total_sectors, length, backing_length;
> int n, ro, open_flags;
> int ret = 0;
> - uint8_t *buf;
> + uint8_t *buf = NULL;
> char filename[PATH_MAX];
>
> if (!drv)
> @@ -1904,7 +1904,24 @@ int bdrv_commit(BlockDriverState *bs)
> }
> }
>
> - total_sectors = bdrv_getlength(bs) >> BDRV_SECTOR_BITS;
> + length = bdrv_getlength(bs);
> + backing_length = bdrv_getlength(bs->backing_hd);
> +
> + if (length < 0 || backing_length < 0) {
> + goto ro_cleanup;
> + }
ret is initialised to 0, so this returns success despite not having
committed anything to the backing file.
Kevin
- [Qemu-devel] [PATCH v4 0/3] block: commits of snapshots larger than backing files, Jeff Cody, 2014/01/23
- [Qemu-devel] [PATCH v4 1/3] block: resize backing file image during offline commit, if necessary, Jeff Cody, 2014/01/23
- Re: [Qemu-devel] [PATCH v4 1/3] block: resize backing file image during offline commit, if necessary,
Kevin Wolf <=
- [Qemu-devel] [PATCH v4 2/3] block: resize backing image during active layer commit, if needed, Jeff Cody, 2014/01/23
- [Qemu-devel] [PATCH v4 3/3] block: update block commit documentation regarding image truncation, Jeff Cody, 2014/01/23