From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Bottomley Subject: Re: [PATCH] tpm-emulator: add a TPM emulator pass through Date: Sun, 15 Jan 2017 22:37:46 -0800 Message-ID: <1484548666.2405.34.camel@HansenPartnership.com> References: <1483923513.2644.1.camel@HansenPartnership.com> <20170109154945.GA28023@obsidianresearch.com> <1483978982.2448.7.camel@HansenPartnership.com> <20170109165416.GA13960@obsidianresearch.com> <1483981445.2398.4.camel@HansenPartnership.com> <20170109171430.GA18648@obsidianresearch.com> <7bef4616-cd69-2798-fc1f-f7eee2fb8c98@linux.vnet.ibm.com> <1483986287.2398.5.camel@HansenPartnership.com> <1e9d8540-63b9-e6fe-d643-30705030d49c@linux.vnet.ibm.com> <1483987877.2398.9.camel@HansenPartnership.com> <1483989503.2398.13.camel@HansenPartnership.com> <1483992413.2398.16.camel@HansenPartnership.com> <7fa906c5-081f-f095-6730-dfcb35cda661@linux.vnet.ibm.com> <1484509202.2405.18.camel@HansenPartnership.com> <2b98d20d-3321-a986-f4f5-a0bd9add6244@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <2b98d20d-3321-a986-f4f5-a0bd9add6244-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: tpmdd-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: Stefan Berger , Jason Gunthorpe Cc: tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: tpmdd-devel@lists.sourceforge.net On Sun, 2017-01-15 at 21:25 -0500, Stefan Berger wrote: > On 01/15/2017 02:40 PM, James Bottomley wrote: > > On Sun, 2017-01-15 at 14:18 -0500, Stefan Berger wrote: > > > On 01/09/2017 03:06 PM, James Bottomley wrote: > > > > > > > > > > Thanks. I fixed this now to make it at least compilable. The > > > > > more > > > > > recent branch is the tpm2-preview.rev138, which makes that > > > > > other > > > > > branch more or less obsolete. > > > > OK, with this branch, I can get a mostly successful run of the > > > > smoke tests. The failure looks to be a dispute over who > > > > handles > > > > TPM_RC_RETRY > > > The following now also works after a (forced) updated on the > > > swtpm > > > TPM2 preview branch: > > > > > > swtpm chardev --vtpm-proxy --tpmstate dir=/tmp --tpm2 > > > swtpm chardev --vtpm-proxy --tpmstate dir=/tmp > > What would it take to get this into the master branch and released, > > so > > those of us who use standard distribution tools can get it? > > The problem lies in the code that writes the TPM 2 state in host > format > rather than in Big Endian format, which would make the state portable > and likely that is the preferred format for QEMU integration. Besides > that there is some functionality missing for suspending the volatile > state of the TPM 2. > > > https://github.com/stefanberger/libtpms/wiki Yes, but my point is that most people who want to be using tpm2 emulation don't really care about QEMU ... it's a nice thing, sure, but it's very complex to set up, so it's only going to be the odd enthusiast that cares about vtpm in QEMU. Most people simply care about testing TPM2 code or playing with TPM2 itself and for that, they don't need or want virtualization. Could you not push tpm2 into the release branch with the caveat that the extremely esoteric BE on LE virtualization case doesn't work yet? James ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi