On 09/25/14 19:36, Eric Blake wrote: > On 09/25/2014 10:40 AM, Peter Krempa wrote: >> Since 363e9a68 we track backing chain metadata when creating snapshots >> the right way even for the inactive configuration. As we did not yet >> update other code paths that modify the backing chain (blockpull) the >> newDef backing chain gets out of sync. >> >> After stopping of a VM the new definition gets copied to the next start >> one. The new VM then has incorrect backing chain info. This patch >> switches the backing chain detector to always purge the existing backing >> chain and forces re-detection to avoid this issue until we'll have full >> backing chain tracking support. >> >> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1144922 >> --- >> src/qemu/qemu_domain.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > ACK. It's a bandaid that may disappear later, but it's better to ship > something inefficient and working rather than efficient but buggy. I was actually surprised the snapshot code tracks the backing chain this well. It will be cool to have full tracking support. I've pushed this now so that we don't present the user with incorrect info until we are that far. Thanks Peter
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list