[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch()
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch() |
Date: |
Tue, 09 Aug 2016 16:29:07 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Marc-André Lureau <address@hidden> writes:
> Hi
>
> ----- Original Message -----
>> address@hidden writes:
>>
>> > From: Marc-André Lureau <address@hidden>
>> >
>> > Replace the old manual dispatch and validation code by the generic one
>> > provided by qapi common code.
>> >
>> > Note that it is now possible to call the following commands that used to
>> > be disabled by compile-time conditionals:
>> > - dump-skeys
>> > - query-spice
>> > - rtc-reset-reinjection
>> > - query-gic-capabilities
>> >
>> > Their fallback functions return an appropriate "feature disabled" error.
>> >
>> > Signed-off-by: Marc-André Lureau <address@hidden>
>>
>> Means query-qmp-schema no longer shows whether these commands are
>> supported, doesn't it?
>>
>> Eric, could this create difficulties for libvirt or other introspection
>> users?
>
> Thinking a bit about this, I guess it would be fairly straightforward
> to have a new key "c-conditional" : "#ifdef CONFIG_SPICE" that would
> prepend it in C generated files, with a corresponding "#endif". Would
> that be acceptable?
Not exactly pretty, but the only alternative I can think of right now
would be conditional qapi generation, i.e. something like
{ 'if': 'CONFIG_SPICE'
'then': { 'command': 'query-spice', 'returns': 'SpiceInfo' } }
More general, but *much* more work. Let's not go there now.
The value of key 'c-conditional' must be a preprocessing directive that
pairs with #endif. Hmm.
Could make it an expression instead, and call the key just
'conditional'. If given, wrap the code generated for the QAPI
definition in
#if <value of conditional>
...
#endif
Feels cleaner, but to avoid -Wundef warnings, we'd have to say
'defined(CONFIG_SPICE)'.
Thoughts?
- [Qemu-devel] [PATCH v3 09/15] qapi: remove the "middle" mode, (continued)
- [Qemu-devel] [PATCH v3 09/15] qapi: remove the "middle" mode, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 10/15] qapi: check invalid arguments on no-args commands, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 11/15] qmp: update qmp_query_spice fallback, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), marcandre . lureau, 2016/08/08
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), Markus Armbruster, 2016/08/09
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), Daniel P. Berrange, 2016/08/09
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), Marc-André Lureau, 2016/08/09
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(),
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), Marc-André Lureau, 2016/08/09
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), Markus Armbruster, 2016/08/09
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), Marc-André Lureau, 2016/08/09
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), Markus Armbruster, 2016/08/10
- Re: [Qemu-devel] [PATCH v3 12/15] monitor: use qmp_dispatch(), Marc-André Lureau, 2016/08/10
[Qemu-devel] [PATCH v3 13/15] build-sys: remove qmp-commands-old.h, marcandre . lureau, 2016/08/08
[Qemu-devel] [PATCH v3 15/15] qmp-commands.txt: fix some styling, marcandre . lureau, 2016/08/08
[Qemu-devel] [PATCH v3 14/15] Drop qmp-commands.hx, marcandre . lureau, 2016/08/08