* [Qemu-devel] [PATCH 1/4] Qemu-Xen-vTPM: Support for Xen stubdom vTPM command line options
@ 2014-11-02 6:39 Quan Xu
2014-11-04 15:55 ` Eric Blake
0 siblings, 1 reply; 3+ messages in thread
From: Quan Xu @ 2014-11-02 6:39 UTC (permalink / raw)
To: qemu-devel; +Cc: lcapitulino, armbru, Quan Xu, xen-devel
Signed-off-by: Quan Xu <quan.xu@intel.com>
---
configure | 14 ++++++++++++++
hmp.c | 7 +++++++
qapi-schema.json | 17 +++++++++++++++--
qemu-options.hx | 13 +++++++++++--
tpm.c | 7 ++++++-
5 files changed, 53 insertions(+), 5 deletions(-)
diff --git a/configure b/configure
index a9e4d49..d63b8a1 100755
--- a/configure
+++ b/configure
@@ -2942,6 +2942,16 @@ else
fi
##########################################
+# TPM xenstubdoms is only on x86 Linux
+
+if test "$targetos" = Linux && test "$cpu" = i386 -o "$cpu" = x86_64 && \
+ test "$xen" = "yes"; then
+ tpm_xenstubdoms=$tpm
+else
+ tpm_xenstubdoms=no
+fi
+
+##########################################
# attr probe
if test "$attr" != "no" ; then
@@ -4333,6 +4343,7 @@ echo "gcov $gcov_tool"
echo "gcov enabled $gcov"
echo "TPM support $tpm"
echo "libssh2 support $libssh2"
+echo "TPM xenstubdoms $tpm_xenstubdoms"
echo "TPM passthrough $tpm_passthrough"
echo "QOM debugging $qom_cast_debug"
echo "vhdx $vhdx"
@@ -4810,6 +4821,9 @@ if test "$tpm" = "yes"; then
if test "$tpm_passthrough" = "yes"; then
echo "CONFIG_TPM_PASSTHROUGH=y" >> $config_host_mak
fi
+ if test "$tpm_xenstubdoms" = "yes"; then
+ echo "CONFIG_TPM_XENSTUBDOMS=y" >> $config_host_mak
+ fi
fi
echo "TRACE_BACKENDS=$trace_backends" >> $config_host_mak
diff --git a/hmp.c b/hmp.c
index 63d7686..5d2bcf9 100644
--- a/hmp.c
+++ b/hmp.c
@@ -689,6 +689,7 @@ void hmp_info_tpm(Monitor *mon, const QDict *qdict)
Error *err = NULL;
unsigned int c = 0;
TPMPassthroughOptions *tpo;
+ TPMXenstubdomsOptions *txo;
info_list = qmp_query_tpm(&err);
if (err) {
@@ -718,6 +719,12 @@ void hmp_info_tpm(Monitor *mon, const QDict *qdict)
tpo->has_cancel_path ? ",cancel-path=" : "",
tpo->has_cancel_path ? tpo->cancel_path : "");
break;
+ case TPM_TYPE_OPTIONS_KIND_XENSTUBDOMS:
+ txo = ti->options->xenstubdoms;
+ if (!txo) {
+ monitor_printf(mon, "null TPMXenstubdomsOptions error!\n");
+ }
+ break;
case TPM_TYPE_OPTIONS_KIND_MAX:
break;
}
diff --git a/qapi-schema.json b/qapi-schema.json
index 24379ab..23ecb4b 100644
--- a/qapi-schema.json
+++ b/qapi-schema.json
@@ -2853,10 +2853,11 @@
# An enumeration of TPM types
#
# @passthrough: TPM passthrough type
+# @xenstubdoms: TPM xenstubdoms type
#
# Since: 1.5
##
-{ 'enum': 'TpmType', 'data': [ 'passthrough' ] }
+{ 'enum': 'TpmType', 'data': [ 'passthrough', 'xenstubdoms' ] }
##
# @query-tpm-types:
@@ -2884,6 +2885,16 @@
{ 'type': 'TPMPassthroughOptions', 'data': { '*path' : 'str',
'*cancel-path' : 'str'} }
+# @TPMXenstubdomsOptions:
+#
+# Information about the TPM xenstubdoms type
+#
+# Since: > 2.1.0
+##
+{ 'type': 'TPMXenstubdomsOptions', 'data': { } }
+#
+##
+
##
# @TpmTypeOptions:
#
@@ -2894,7 +2905,9 @@
# Since: 1.5
##
{ 'union': 'TpmTypeOptions',
- 'data': { 'passthrough' : 'TPMPassthroughOptions' } }
+ 'data': { 'passthrough' : 'TPMPassthroughOptions',
+ 'xenstubdoms' : 'TPMXenstubdomsOptions' } }
+##
##
# @TpmInfo:
diff --git a/qemu-options.hx b/qemu-options.hx
index 1e7d5b8..fd73f57 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -2485,7 +2485,8 @@ DEF("tpmdev", HAS_ARG, QEMU_OPTION_tpmdev, \
"-tpmdev passthrough,id=id[,path=path][,cancel-path=path]\n"
" use path to provide path to a character device; default is /dev/tpm0\n"
" use cancel-path to provide path to TPM's cancel sysfs entry; if\n"
- " not provided it will be searched for in /sys/class/misc/tpm?/device\n",
+ " not provided it will be searched for in /sys/class/misc/tpm?/device\n"
+ "-tpmdev xenstubdoms,id=id\n",
QEMU_ARCH_ALL)
STEXI
@@ -2495,7 +2496,8 @@ The general form of a TPM device option is:
@item -tpmdev @var{backend} ,id=@var{id} [,@var{options}]
@findex -tpmdev
Backend type must be:
-@option{passthrough}.
+@option{passthrough}, or
+@option{xenstubdoms}.
The specific backend type will determine the applicable options.
The @code{-tpmdev} option creates the TPM backend and requires a
@@ -2545,6 +2547,13 @@ To create a passthrough TPM use the following two options:
Note that the @code{-tpmdev} id is @code{tpm0} and is referenced by
@code{tpmdev=tpm0} in the device option.
+To create a xenstubdoms TPM use the following two options:
+@example
+-tpmdev xenstubdoms,id=tpm0 -device tpm-tis,tpmdev=tpm0
+@end example
+Note that the @code{-tpmdev} id is @code{tpm0} and is referenced by
+@code{tpmdev=tpm0} in the device option.
+
@end table
ETEXI
diff --git a/tpm.c b/tpm.c
index c371023..ee9acb8 100644
--- a/tpm.c
+++ b/tpm.c
@@ -25,7 +25,7 @@ static QLIST_HEAD(, TPMBackend) tpm_backends =
#define TPM_MAX_MODELS 1
-#define TPM_MAX_DRIVERS 1
+#define TPM_MAX_DRIVERS 2
static TPMDriverOps const *be_drivers[TPM_MAX_DRIVERS] = {
NULL,
@@ -256,6 +256,7 @@ static TPMInfo *qmp_query_tpm_inst(TPMBackend *drv)
{
TPMInfo *res = g_new0(TPMInfo, 1);
TPMPassthroughOptions *tpo;
+ TPMXenstubdomsOptions *txo;
res->id = g_strdup(drv->id);
res->model = drv->fe_model;
@@ -275,6 +276,10 @@ static TPMInfo *qmp_query_tpm_inst(TPMBackend *drv)
tpo->has_cancel_path = true;
}
break;
+ case TPM_TYPE_XENSTUBDOMS:
+ res->options->kind = TPM_TYPE_OPTIONS_KIND_XENSTUBDOMS;
+ txo = g_new0(TPMXenstubdomsOptions, 1);
+ res->options->xenstubdoms = txo;
case TPM_TYPE_MAX:
break;
}
--
1.8.3.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH 1/4] Qemu-Xen-vTPM: Support for Xen stubdom vTPM command line options
2014-11-02 6:39 [Qemu-devel] [PATCH 1/4] Qemu-Xen-vTPM: Support for Xen stubdom vTPM command line options Quan Xu
@ 2014-11-04 15:55 ` Eric Blake
2014-11-04 16:43 ` Xu, Quan
0 siblings, 1 reply; 3+ messages in thread
From: Eric Blake @ 2014-11-04 15:55 UTC (permalink / raw)
To: Quan Xu, qemu-devel; +Cc: xen-devel, armbru, lcapitulino
[-- Attachment #1: Type: text/plain, Size: 1509 bytes --]
On 11/02/2014 07:39 AM, Quan Xu wrote:
[meta-comment] This message appears to have a cover letter; but you
failed to properly set the 'In-Reply-To' header; when each patch of your
series appears as a top-level thread instead of a reply to the cover
letter, it is harder to review.
> Signed-off-by: Quan Xu <quan.xu@intel.com>
> ---
> configure | 14 ++++++++++++++
> hmp.c | 7 +++++++
> qapi-schema.json | 17 +++++++++++++++--
> qemu-options.hx | 13 +++++++++++--
> tpm.c | 7 ++++++-
> 5 files changed, 53 insertions(+), 5 deletions(-)
>
> +++ b/qapi-schema.json
> @@ -2853,10 +2853,11 @@
> # An enumeration of TPM types
> #
> # @passthrough: TPM passthrough type
> +# @xenstubdoms: TPM xenstubdoms type
Missing a '(since 2.3)' designation.
> #
> # Since: 1.5
> ##
> -{ 'enum': 'TpmType', 'data': [ 'passthrough' ] }
> +{ 'enum': 'TpmType', 'data': [ 'passthrough', 'xenstubdoms' ] }
>
> ##
> # @query-tpm-types:
> @@ -2884,6 +2885,16 @@
> { 'type': 'TPMPassthroughOptions', 'data': { '*path' : 'str',
> '*cancel-path' : 'str'} }
>
> +# @TPMXenstubdomsOptions:
> +#
> +# Information about the TPM xenstubdoms type
> +#
> +# Since: > 2.1.0
2.1 is wrong; the earliest you can get this into the tree is 2.3
(because you've missed soft freeze for 2.2).
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 539 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH 1/4] Qemu-Xen-vTPM: Support for Xen stubdom vTPM command line options
2014-11-04 15:55 ` Eric Blake
@ 2014-11-04 16:43 ` Xu, Quan
0 siblings, 0 replies; 3+ messages in thread
From: Xu, Quan @ 2014-11-04 16:43 UTC (permalink / raw)
To: Eric Blake, qemu-devel@nongnu.org
Cc: xen-devel@lists.xen.org, armbru@redhat.com,
lcapitulino@redhat.com
> -----Original Message-----
> From: Eric Blake [mailto:eblake@redhat.com]
> Sent: Tuesday, November 04, 2014 11:55 PM
> To: Xu, Quan; qemu-devel@nongnu.org
> Cc: lcapitulino@redhat.com; armbru@redhat.com; xen-devel@lists.xen.org
> Subject: Re: [Qemu-devel] [PATCH 1/4] Qemu-Xen-vTPM: Support for Xen
> stubdom vTPM command line options
>
> On 11/02/2014 07:39 AM, Quan Xu wrote:
>
> [meta-comment] This message appears to have a cover letter; but you failed
> to properly set the 'In-Reply-To' header; when each patch of your series
> appears as a top-level thread instead of a reply to the cover letter, it is harder
> to review.
>
:(
Thanks Eric.
Markus (Markus Armbruster <armbru@redhat.com>) shared how he formats and sends a patch series.
I will submit v2 in the right format.
> > Signed-off-by: Quan Xu <quan.xu@intel.com>
> > ---
> > configure | 14 ++++++++++++++
> > hmp.c | 7 +++++++
> > qapi-schema.json | 17 +++++++++++++++-- qemu-options.hx | 13
> > +++++++++++--
> > tpm.c | 7 ++++++-
> > 5 files changed, 53 insertions(+), 5 deletions(-)
> >
>
> > +++ b/qapi-schema.json
> > @@ -2853,10 +2853,11 @@
> > # An enumeration of TPM types
> > #
> > # @passthrough: TPM passthrough type
> > +# @xenstubdoms: TPM xenstubdoms type
>
> Missing a '(since 2.3)' designation.
I will add it in v2.
>
> > #
> > # Since: 1.5
> > ##
> > -{ 'enum': 'TpmType', 'data': [ 'passthrough' ] }
> > +{ 'enum': 'TpmType', 'data': [ 'passthrough', 'xenstubdoms' ] }
> >
> > ##
> > # @query-tpm-types:
> > @@ -2884,6 +2885,16 @@
> > { 'type': 'TPMPassthroughOptions', 'data': { '*path' : 'str',
> > '*cancel-path' : 'str'}
> > }
> >
> > +# @TPMXenstubdomsOptions:
> > +#
> > +# Information about the TPM xenstubdoms type # # Since: > 2.1.0
>
> 2.1 is wrong; the earliest you can get this into the tree is 2.3 (because you've
> missed soft freeze for 2.2).
I will fix it in v2.
>
> --
> Eric Blake eblake redhat com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-11-04 16:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-02 6:39 [Qemu-devel] [PATCH 1/4] Qemu-Xen-vTPM: Support for Xen stubdom vTPM command line options Quan Xu
2014-11-04 15:55 ` Eric Blake
2014-11-04 16:43 ` Xu, Quan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).