[libvirt] [PATCH 0/2] qemu: Enable -blockdev support (blockdev-add saga)

[Daniel P. Berrangé]

On Fri, Sep 13, 2019 at 02:43:53PM +0200, Peter Krempa wrote:
> To my knowledge, everything in libvirt is now prepared to fully use
> -blockdev way to configure disks in qemu.
> 
> There is one known qemu bug though: Internal snapshots don't work with
> -blockdev:
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1658981

What are the chances of that bug actually getting fixed any time in the
forseeable future ?

savevm has been a pain for a long time since it blocks QEMU and we've
been waiting for it to be fixedd by QEMU devs for at least 7-8 years
now, and I don't see anyone from QEMU block maintainers commenting on
this new BZ to indicate they're going to fix this latest problem.

> Since I can't in good faith ask for merging this patchset yet I'd like
> to give it some more testing I'm suggesting that we push it and revert
> it during freeze or add a capability check once qemu is fixed.

I'd only be in favour of enabling it if there is some clear time frame
on whicht hat bug above is likely to be fixed. I'm guesing it is pretty
unlikely to be fixed before our freeze, so I thinking speculatively
enabling it is premature.

> Any other ideas?

We already have the option you added to allow capabilities to be
force enabled for people to test with, though admittedly that
is going to get fairly limited usage.



Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|