[libvirt] [PATCH 2/3] qemu: qapi: Implement support for 'features'

Peter Krempa pkrempa at redhat.com
Wed Jun 19 10:32:23 UTC 2019


On Wed, Jun 19, 2019 at 12:31:18 +0200, Jiri Denemark wrote:
> On Tue, Jun 18, 2019 at 10:37:25 +0200, Peter Krempa wrote:
> > Starting from version 4.1 qemu allows reporting 'features' for a given
> > QAPI type object. This allows reporting support of fixes and additions
> > which are otherwise invisible in the QAPI schema.
> > 
> > Implement a possibility to query 'features' in the QAPI query strings.
> > 
> > Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> > ---
> >  src/qemu/qemu_qapi.c | 41 +++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 41 insertions(+)
> > 
> > diff --git a/src/qemu/qemu_qapi.c b/src/qemu/qemu_qapi.c
> > index 4ed67b68bc..5e6dab4417 100644
> > --- a/src/qemu/qemu_qapi.c
> > +++ b/src/qemu/qemu_qapi.c
> > @@ -109,6 +109,38 @@ virQEMUQAPISchemaTraverse(const char *baseName,
> >                            struct virQEMUQAPISchemaTraverseContext *ctxt);
> > 
> > 
> > +/**
> > + * @featurename: name of 'feature' field to select
> > + * @elem: QAPI JSON entry for a type
> > + *
> > + * Looks for @featurename in the array of 'features' for given type passed in
> > + * via @elem. Returns the pointer to the JSON string representing @feature.
> 
> Hmm, the function actually returns some magic 0, 1, or -2 values rather
> than a pointer to anything. Ah, I see the values are documented for
> virQEMUQAPISchemaTraverseFunc function pointer so the values are not
> really magic.

Oops, right I forgot to fix docs after I changed the approach how to do
it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190619/c07b23e5/attachment-0001.sig>


More information about the libvir-list mailing list