Re: [Qemu-devel] [PATCH 4/4] tpm: Provide libtpms software TPM backend

[Corey Bryant <coreyb@linux.vnet.ibm.com>]

On 11/25/2013 10:04 PM, Xu, Quan wrote:
>       Thanks Bryant, this problem has been solved by following "http://www.mail-archive.com/qemu-devel@nongnu.org/msg200808.html".
>       But there is another problem when run configure with "./configure --target-list=x86_64-softmmu --enable-tpm". The value of "libtpms" is still "no". when I modified "tpm_libtpms" to "yes" in configure file directly and make, then reported with error
> "hw/tpm/tpm_libtpms.c:21:33: fatal error: libtpms/tpm_library.h: No such file or directory".  Now I am installing libtpms with https://github.com/coreycb/libtpms for libtpms lib. Could you share specific step to configure QEMU based on your patch, if it comes
> easily to you?

Here's what I've been using to build libtpms:

$ CFLAGS='-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions 
-fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic'
$ export CFLAGS
$ ./configure --build=x86_64-redhat-linux-gnu --prefix=/usr 
--libdir=/usr/lib64
$ make
$ sudo make install

And then the configure you're using above should work for QEMU.

>       BTW, one target of my team is enabling stubdom vtpm for HVM virtual machine on Xen virtualization, your patches and seabios are big breakthroughs. My team is very interested to collaborate with you / Qemu community on similar areas.

That's great to hear!

Unfortunately, the current approach of linking QEMU against libtpms 
doesn't look like it's going to make it upstream.  So it looks like we 
need to take a different approach.

Btw, I thought Xen already had TPM support.  Is that not supported in 
stubdom's?

-- 
Regards,
Corey Bryant

>
> I'd be really pleased if you can help me on these issues.
>
> Quan Xu
> Intel
>
>
>> -----Original Message-----
>> From: Corey Bryant [mailto:coreyb@linux.vnet.ibm.com]
>> Sent: Monday, November 25, 2013 9:53 PM
>> To: Xu, Quan
>> Cc: qemu-devel@nongnu.org
>> Subject: Re: [Qemu-devel] [PATCH 4/4] tpm: Provide libtpms software TPM
>> backend
>>
>>
>>
>> On 11/24/2013 10:36 PM, Xu, Quan wrote:
>>> Bryant,
>>>
>>>        I found that there is some conflict in qemu-options.hx between your
>> patch andqemu-1.7.0-rc1.tar.bz2
>> <http://wiki.qemu-project.org/download/qemu-1.7.0-rc1.tar.bz2>.
>>>
>>> What QEMU version does this patch base on? Thanks.
>>>
>>>
>>>
>>>
>>>
>>> Quan Xu
>>>
>>> Intel
>>>
>>>
>>>
>>
>> Thanks Quan.  I believe I built these on top of commit
>> c2d30667760e3d7b81290d801e567d4f758825ca.  I don't think this series is going
>> to make it upstream though so I likely won't be submitting a v2.
>>
>> --
>> Regards,
>> Corey Bryant
>
>