On 10/04/2016 01:22 PM, Eduardo Habkost wrote:
On Mon, Oct 03, 2016 at 03:15:39PM -0500, Eric Blake wrote:
[...]
>> 3.2) Removing the unimplemented command from query-commands only
>> (by calling qmp_disable_command()), but keeping it on the QAPI
>> schema. I am not sure it's OK to do that. If it is, this
>> sounds like the simplest solution.
>
> We already have existing commands in this category, and it is
> conceptually the easiest (if query-commands doesn't list the command,
> then the command doesn't work even if the introspection still shows it).
[...]
Which commands are already in this category? I don't see any
qmp_disable_command() calls on the tree except for guest agent
code.
Probably a typo on my side; commit 5032a16d adds the use of
qmp_unregister_command(), not qmp_disable_command().
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org