[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/9] qmp: Clean up qmp_query_spice() #ifndef !CO
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 4/9] qmp: Clean up qmp_query_spice() #ifndef !CONFIG_SPICE dummy |
Date: |
Wed, 14 Jan 2015 06:26:16 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 01/13/2015 10:50 AM, Markus Armbruster wrote:
> QMP command query-spice exists only #ifdef CONFIG_SPICE. Due to QAPI
> limitations, we need a dummy function anyway, but it's unreachable.
>
> Our current dummy function goes out of its way to produce the exact
> same error as the QMP core does for unknown commands. Cute, but both
> unclean and unnecessary. Replace by straight abort().
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> + * qmp-commands.hx ensures that QMP command query-spice exists only
> + * #ifdef CONFIG_SPICE. Necessary for an accurate query-commands
> + * result. However, the QAPI schema is blissfully unaware of that,
> + * and the QAPI code generator happily generates a dead
> + * qmp_marshal_input_query_spice() that calls qmp_query_spice().
> + * Provide it one, or else linking fails.
> + * FIXME Educate the QAPI schema on CONFIG_SPICE.
There's probably several commands that are only conditionally compiled
in qmp-commands.hx, but where the qapi has no way to express that they
are conditional on how the binary was compiled. I'm not sure if it
would help the user to document that a command listed in the .json might
not actually exist, but if enhancing the qapi code generator to
understand conditionals made for less maintenance, then that alone would
be worth cleaning up this FIXME.
In the meantime, your patch is just fine.
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 0/9] qmp hmp balloon: Cleanups around error reporting, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 1/9] qmp hmp: Factor out common "using spice" test, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 4/9] qmp: Clean up qmp_query_spice() #ifndef !CONFIG_SPICE dummy, Markus Armbruster, 2015/01/13
- Re: [Qemu-devel] [PATCH 4/9] qmp: Clean up qmp_query_spice() #ifndef !CONFIG_SPICE dummy,
Eric Blake <=
- [Qemu-devel] [PATCH 3/9] hmp: Compile hmp_info_spice() only with CONFIG_SPICE, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 6/9] qmp: Eliminate silly QERR_COMMAND_NOT_FOUND macro, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 2/9] qmp hmp: Improve error messages when SPICE is not in use, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 7/9] balloon: Inline qemu_balloon(), qemu_balloon_status(), Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 8/9] balloon: Factor out common "is balloon active" test, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 9/9] balloon: Eliminate silly QERR_ macros, Markus Armbruster, 2015/01/13