[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 28/29] Fix up dangling references to qmp-comm
From: |
Michael Roth |
Subject: |
Re: [Qemu-devel] [PATCH v2 28/29] Fix up dangling references to qmp-commands.* in comment and doc |
Date: |
Mon, 19 Feb 2018 13:38:21 -0600 |
User-agent: |
alot/0.6 |
Quoting Markus Armbruster (2018-02-11 03:36:06)
> Fix up the reference to qmp-commands.hx in qmp.c. Missed in commit
> 5032a16d1d.
>
> Fix up the reference to qmp-commands.txt in
> docs/xen-save-devices-state.txt. Missed in commit 4d8bb958fa.
>
> Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Michael Roth <address@hidden>
> ---
> docs/xen-save-devices-state.txt | 3 +--
> qmp.c | 14 +++++++-------
> 2 files changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/docs/xen-save-devices-state.txt b/docs/xen-save-devices-state.txt
> index a72ecc8081..1912ecad25 100644
> --- a/docs/xen-save-devices-state.txt
> +++ b/docs/xen-save-devices-state.txt
> @@ -8,8 +8,7 @@ These operations are normally used with migration (see
> migration.txt),
> however it is also possible to save the state of all devices to file,
> without saving the RAM or the block devices of the VM.
>
> -This operation is called "xen-save-devices-state" (see
> -qmp-commands.txt)
> +The save operation is available as QMP command xen-save-devices-state.
>
>
> The binary format used in the file is the following:
> diff --git a/qmp.c b/qmp.c
> index a8d4eba973..ba82e1df9f 100644
> --- a/qmp.c
> +++ b/qmp.c
> @@ -147,13 +147,13 @@ VncInfo2List *qmp_query_vnc_servers(Error **errp)
>
> #ifndef CONFIG_SPICE
> /*
> - * 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_query_spice() that calls qmp_query_spice(). Provide it
> - * one, or else linking fails. FIXME Educate the QAPI schema on
> - * CONFIG_SPICE.
> + * qmp_unregister_commands_hack() 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_query_spice() that calls qmp_query_spice().
> + * Provide it one, or else linking fails. FIXME Educate the QAPI
> + * schema on CONFIG_SPICE.
> */
> SpiceInfo *qmp_query_spice(Error **errp)
> {
> --
> 2.13.6
>
- Re: [Qemu-devel] [PATCH v2 10/29] qapi: Touch generated files only when they change, (continued)
Re: [Qemu-devel] [PATCH v2 10/29] qapi: Touch generated files only when they change, Michael Roth, 2018/02/19
[Qemu-devel] [PATCH v2 26/29] docs: Correct outdated information on QAPI, Markus Armbruster, 2018/02/11
[Qemu-devel] [PATCH v2 28/29] Fix up dangling references to qmp-commands.* in comment and doc, Markus Armbruster, 2018/02/11
[Qemu-devel] [PATCH v2 29/29] qapi: Don't create useless directory qapi-generated, Markus Armbruster, 2018/02/11
[Qemu-devel] [PATCH v2 09/29] qapi-gen: Convert from getopt to argparse, Markus Armbruster, 2018/02/11
[Qemu-devel] [PATCH v2 20/29] qapi/types qapi/visit: Generate built-in stuff into separate files, Markus Armbruster, 2018/02/11