qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH V20 1/8] Support for TPM command line options


From: Stefan Berger
Subject: Re: [Qemu-devel] [PATCH V20 1/8] Support for TPM command line options
Date: Thu, 07 Feb 2013 07:00:40 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120911 Thunderbird/15.0.1

On 02/04/2013 06:21 AM, Luiz Capitulino wrote:
On Fri, 01 Feb 2013 10:33:01 -0500
Corey Bryant <address@hidden> wrote:

+##
+# @TPMInfo:
+#
+# Information about the TPM
+#
+# @model: The TPM frontend model, i.e., tpm-tis
+#
+# @id: The ID of the TPM
+#
+# @type: The type of TPM backend, i.e., passthrough
+#
+# @path: #optional Path to the TPM backend device
+#
+# @cancel_path: #optional Path to TPM backend device's cancel sysfs entry
+#
+# Since: 1.5.0
+##
+{ 'type': 'TPMInfo',
+  'data': {'model': 'str', 'id': 'str', 'type': 'str', '*path': 'str',
+           '*cancel_path': 'str' } }
+
It might be preferred that you break the monitor support into it's own
patch, and logically it would make more sense to introduce these backend
specific members after the backend patch.  But those are just nit comments.

More importantly, I have a question (probably for Luiz) regarding future
modification of the monitor command.  In the future, if a new vTPM
backend is introduced (e.g. an emulated software vTPM), can we add new
optional members to the end of this TPMInfo?  For example, could we
modify it to this in the future?
Usually, extensions like that are only allowed for query- commands. For
non-query commands we prefer adding new commands instead.

For 'query-tpm' we should be fine then.


Btw, we avoid using free strings like 'type' and 'model', please use
an enumeration instead.

Ok, did that.


PS: I wonder if it would be possible to extend enumerations and unions
     though.


Regards,
   Stefan




reply via email to

[Prev in Thread] Current Thread [Next in Thread]