[libvirt] [PATCH] qemu: driver: Allow passing disk target as top image with block commit

[Eric Blake]

On 05/17/2017 10:20 AM, Peter Krempa wrote:
> Since we allow active layer block commit the users are allowed to commit
> the top of the chain (e.g. vda) into the backing image. The API would
> not accept that parameter, as it tried to look up the image in the
> backing chain.
> 
> Add the ability to use the top level image target name explicitly as the
> top image of the block commit operation.
> 
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1451394
> ---
>  src/qemu/qemu_driver.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

ACK

> 
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 1c4873e48..36d8a701a 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -16942,7 +16942,7 @@ qemuDomainBlockCommit(virDomainPtr dom,
>      if (qemuDomainDiskBlockJobIsActive(disk))
>          goto endjob;
> 
> -    if (!top)
> +    if (!top || STREQ(top, disk->dst))
>          topSource = disk->src;
>      else if (virStorageFileParseChainIndex(disk->dst, top, &topIndex) < 0 ||
>               !(topSource = virStorageFileChainLookup(disk->src, NULL,
> 

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170517/ca63c953/attachment-0001.sig>