On 12/06/2016 10:07 AM, Eric Blake wrote:
On 12/06/2016 10:00 AM, Peter Krempa wrote:
> Qemu 2.8.0+ changes arguments structure for blockdev-add in the effort
> to make it finally stable. Since libvirt recently added the detection of
> gluster debug support relying on the old syntax we need to add the new
> as well.
> ---
> src/qemu/qemu_capabilities.c | 1 +
> tests/qemucapabilitiesdata/caps_2.8.0.x86_64.xml | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
> index df417f2..214f848 100644
> --- a/src/qemu/qemu_capabilities.c
> +++ b/src/qemu/qemu_capabilities.c
> @@ -1725,6 +1725,7 @@ static struct virQEMUCapsStringFlags
virQEMUCapsObjectPropsUSBNECXHCI[] = {
> /* see documentation for virQEMUCapsQMPSchemaGetByPath for the query format */
> static struct virQEMUCapsStringFlags virQEMUCapsQMPSchemaQueries[] = {
> { "blockdev-add/arg-type/options/+gluster/debug-level",
QEMU_CAPS_GLUSTER_DEBUG_LEVEL},
> + { "blockdev-add/arg-type/+gluster/debug-level",
QEMU_CAPS_GLUSTER_DEBUG_LEVEL},
Except that in 2.8, it will be spelled 'debug', not 'debug-level':
https://lists.gnu.org/archive/html/qemu-devel/2016-12/msg00572.html
https://lists.gnu.org/archive/html/qemu-devel/2016-12/msg00567.html
debug-level should only be used in 2.7.
More on this:
qemu 2.7 command line spells it "debug", it was only QMP that spelled it
"debug-level". And libvirt's only use of QEMU_CAPS_GLUSTER_DEBUG_LEVEL
was to request "debug" from the command line. Eventually, libvirt will
want to use blockdev-add, and we'd prefer that the spelling be
consistent throughout (that is, command line and hotplug through QMP
should be identical). Which is why qemu 2.8 is fixing the QMP spelling
to match the command line for -rc3.
So Jeff's patch to qemu 2.8 is still correct, and all we need on the
libvirt side is to match the patch spelling to the actual qemu 2.8
release (and not the 2.8-rc2 release).
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org