[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/12] qapi-schema: use generated marshaller for
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 01/12] qapi-schema: use generated marshaller for 'qmp_capabilities' |
Date: |
Thu, 23 Jun 2016 16:41:05 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 06/22/2016 06:07 PM, address@hidden wrote:
> From: Marc-André Lureau <address@hidden>
>
> qapi'fy the 'qmp_capabilities' command.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> monitor.c | 4 ++--
> qapi-schema.json | 19 +++++++++++++++++++
> qmp-commands.hx | 2 +-
> 3 files changed, 22 insertions(+), 3 deletions(-)
>
> +++ b/qapi-schema.json
> @@ -20,6 +20,25 @@
> # QAPI introspection
> { 'include': 'qapi/introspect.json' }
>
> +##
> +# @qmp_capabilities:
> +#
> +# Enable QMP capabilities.
> +#
> +# Arguments: None.
> +#
> +# Example:
> +#
> +# -> { "execute": "qmp_capabilities" }
> +# <- { "return": {} }
> +#
> +# Notes: This command is valid exactly when first connecting: it must be
> +# issued before any other command will be accepted, and will fail once the
> +# monitor is accepting other commands.
> +#
Missing a 'Since: x.y' designation. Part of me argues it would be 2.7,
since that is when it shows up in introspection, part of me argues it
would be 0.13, since it has always been part of QMP. Or maybe we just
omit it? Anyone else with a favorite bikeshed color?
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 00/12] qapi: remove the 'middle' mode, marcandre . lureau, 2016/06/22
- [Qemu-devel] [PATCH 02/12] qapi-schema: add 'device_add', marcandre . lureau, 2016/06/22
- [Qemu-devel] [PATCH 01/12] qapi-schema: use generated marshaller for 'qmp_capabilities', marcandre . lureau, 2016/06/22
- Re: [Qemu-devel] [PATCH 01/12] qapi-schema: use generated marshaller for 'qmp_capabilities',
Eric Blake <=
- [Qemu-devel] [PATCH 05/12] monitor: register the qapi generated commands, marcandre . lureau, 2016/06/22
- [Qemu-devel] [PATCH 07/12] monitor: implement 'qmp_query_commands' without qmp_cmds, marcandre . lureau, 2016/06/22
- [Qemu-devel] [PATCH 06/12] monitor: remove mhandler.cmd_new, marcandre . lureau, 2016/06/22