Re: [PATCH v3 0/5] Active commit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/11/2014 01:53 PM, Adam Litke wrote:
> I've tested this with the following script and it seems that
> specifying base and top explicitly does not work.  Using --shallow
> does work for me though.  Running the included script yields the
> following output:
> 
> $ sudo bash /home/alitke/test/test.sh Formatting 'base.img', fmt=raw

> 
> error: invalid argument: could not find image '/tmp/snap2.img' in
> chain for '/tmp/snap2.img'


> 
> base=$PWD/snap1.img
> top=$PWD/snap2.img
> virsh blockcommit testvm1 vda 0 $base $top --wait --verbose --pivot

Okay, I see the flaw - when looking up an explicit top, we aren't
starting from the active layer.  Try omitting the --top argument as a
workaround.  Meanwhile, this patch should fix it:

diff --git i/src/qemu/qemu_driver.c w/src/qemu/qemu_driver.c
index 60dc9c6..aded8e0 100644
--- i/src/qemu/qemu_driver.c
+++ w/src/qemu/qemu_driver.c
@@ -15575,7 +15575,7 @@ qemuDomainBlockCommit(virDomainPtr dom,
         topSource = disk->src;
     else if (virStorageFileParseChainIndex(disk->dst, top, &topIndex) <
0 ||
              !(topSource = virStorageFileChainLookup(disk->src,
-
disk->src->backingStore,
+                                                     disk->src,
                                                      top, topIndex,
                                                      &top_parent)))
         goto endjob;


-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]