[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/4] tpm: QMP/HMP support for libtpms TPM backen
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 3/4] tpm: QMP/HMP support for libtpms TPM backend |
Date: |
Wed, 06 Nov 2013 09:13:17 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 |
On 11/06/2013 07:39 AM, Corey Bryant wrote:
[your git settings are odd; your messages came across as individual
threads rather than in-reply to the cover letter]
> This patch provides HMP 'info tpm', QMP 'query-tpm' and QMP
> 'query-tpm-types' support for the libtpms TPM backend.
>
> Signed-off-by: Corey Bryant <address@hidden>
> ---
> hmp.c | 5 +++++
> include/sysemu/tpm_backend.h | 1 +
> qapi-schema.json | 18 ++++++++++++++++--
> tpm.c | 7 +++++++
> 4 files changed, 29 insertions(+), 2 deletions(-)
>
> +++ b/qapi-schema.json
> @@ -3673,10 +3673,11 @@
> # An enumeration of TPM types
> #
> # @passthrough: TPM passthrough type
> +# @libtpms: TPM libtpms type
Worth adding '(since 1.8)' to mark when this enum value was added.
> # @TpmTypeOptions:
> #
> # A union referencing different TPM backend types' configuration options
> #
> # @passthrough: The configuration options for the TPM passthrough type
> +# @libtpms: The configuration options for the TPM libtpms type
here as well.
> #
> # Since: 1.5
> ##
> { 'union': 'TpmTypeOptions',
> - 'data': { 'passthrough' : 'TPMPassthroughOptions' } }
> + 'data': { 'passthrough' : 'TPMPassthroughOptions',
> + 'libtpms' : 'TPMLibtpmsOptions' } }
>
Otherwise it looks okay to me.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature