Re: [Qemu-devel] [PATCH v10 09/28] qmp: add query-sev command

[Eric Blake <eblake@redhat.com>]

On 02/28/2018 03:10 PM, Brijesh Singh wrote:
> The QMP query command can used to retrieve the SEV information when
> memory encryption is enabled on AMD platform.
> 
> Cc: Eric Blake <eblake@redhat.com>
> Cc: "Daniel P. Berrangé" <berrange@redhat.com>
> Cc: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> Cc: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Brijesh Singh <brijesh.singh@amd.com>
> ---
>   monitor.c             |  7 ++++++
>   qapi-schema.json      | 61 +++++++++++++++++++++++++++++++++++++++++++++++++++

Based on the pull request I just sent:
https://lists.gnu.org/archive/html/qemu-devel/2018-03/msg00394.html
you will have to rebase this to stick things in qapi/misc.json instead.

>   target/i386/monitor.c |  8 +++++++
>   3 files changed, 76 insertions(+)
> 

> +++ b/qapi-schema.json
> @@ -3189,3 +3189,64 @@
>   # Since: 2.11
>   ##
>   { 'command': 'watchdog-set-action', 'data' : {'action': 'WatchdogAction'} }
> +
> +##
> +# @SevState:
> +#
> +# An enumeration of SEV state information used during @query-sev.
> +#
> +# Since: 2.12
> +##
> +{ 'enum': 'SevState',
> +  'data': ['uninit', 'lupdate', 'lsecret', 'running', 'supdate', 'rupdate' ] }

Could perhaps give more details about the meaning of each state.  But 
I'm not seeing anything blatantly wrong with the UI changes.

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org