From: Matthew Fioravante <matthew.fioravante@jhuapl.edu>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: [PATCH] Upgrade vtpmd to berlios version 0.7.4
Date: Tue, 18 Sep 2012 13:33:48 -0400 [thread overview]
Message-ID: <5058B07C.1070404@jhuapl.edu> (raw)
In-Reply-To: <1347953926.25803.89.camel@dagon.hellion.org.uk>
[-- Attachment #1.1.1: Type: text/plain, Size: 3509 bytes --]
On 09/18/2012 03:38 AM, Ian Campbell wrote:
> On Mon, 2012-09-17 at 18:52 +0100, Matthew Fioravante wrote:
>> What will follow soon are updates to vtpmd, vtpm_manager, xm, xl,
>> mini-os, and new vtpm and vtpm manager stub domains.
> No need to update xm any more, it is deprecated in 4.2.
I have just one small patch for xm coming. You can decide if you want to
accept or not. Its a small addition.
>
> Who or what is/are Berlios? Are they (actively) maintaining the vTPM?
The berlios tpm emulator is what vtpm is based off of.
http://tpm-emulator.berlios.de/
Essentially vtpmd and vtpm-stubdom (coming in a patch) are just small
patches for the tpm emulator to allow it to run in the vtpm framework.
The older version of the emulator required more invasive patching. The
newer one lets you set some function pointers to override functionality
such as saving and loading to disk, making it much easier and simpler to
modify it to become a vtpm.
>
> Perhaps we should consider making this stuff an external dependency
> rather than importing it into our code base? This could be done either
> as a simple dependency (i.e. require vtpm to be installed before
> building) or as a repo cloned during build (like how we handle qemu and
> seabios etc).
>
This might be workable. The vtpm system has 2 mini-so stubdoms and 3
mini-os tpm drivers. Does mini-os/stubdom have a nice way of building
stuff out of tree? It doesn't appear so at the moment. The whole stubdom
setup is basically one large monolithic makefile. vtpm domains also
require building openssl, polarssl, and libgmp in the stubdom cross root.
Should the tpm drivers should be included in mini-os for potential use
by other projects or also kept in their own separate tree? They are GPL
drivers so maybe out of tree makes more sense.
vtpmd and vtpm_managerdom (deprecated, the old way of running vtpms as
processes in dom0, instead of separate domains) could easily be moved
out of tree. They are just binaries installed on the system.
vtpm support for xm/xl probably has to stay in xen. Unless someone plans
on making a plugin architecture for xl. There are also the hotplug
scripts, but those go away with xl.
>> The first patch I'd like to submit upgrades vtpmd to version 0.7.4
>>
>> This patch does the following:
>> -add checks to configure to check for cmake (required by berlios 0.7.4)
> Is the model with cmake that it is required on all the end systems
> building the project (like make), or is it only needed on the project
> maintainer's system (like autoconf/make)?
Its the equivalent of running a configure script, so its needed on end
systems. Also you have to download and patch the tpm emulator before
running cmake, so if we did want to avoid its usage on end systems we
would have to ship the entire source code of the patched emulator.
>
>> -removes all of the 0.5.1 patches
>> -adds a single patch for 0.7.4
>> -cleans up the makefile, should work for parallel make (avoiding
>> version.h discussion from august 2012)
>> -builds vtpmd to use berlios 0.7.4
>> -Remoed the tpm_emualtor build option. berlios itself provides a kernel
>> module if you want to use it in dom0 to emulate the physical tpm.
> Is there going to be an associated documentation update/refresh?
Right now I don't have a formal doc but I want to write one. I'm looking
into getting some time to do this.
>
>> Signed of by: Matthew Fioravante matthew.fioravante@jhuapl.edu
> Ian.
>
[-- Attachment #1.1.2: Type: text/html, Size: 5409 bytes --]
[-- Attachment #1.2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 1459 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2012-09-18 17:33 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-17 17:52 [PATCH] Upgrade vtpmd to berlios version 0.7.4 Matthew Fioravante
2012-09-18 7:38 ` Ian Campbell
2012-09-18 17:33 ` Matthew Fioravante [this message]
2012-09-25 9:53 ` Ian Campbell
2012-09-25 15:50 ` Matthew Fioravante
2012-09-26 11:46 ` George Dunlap
2012-09-26 14:39 ` Matthew Fioravante
2012-09-26 15:09 ` George Dunlap
2012-09-26 15:21 ` Ian Campbell
2012-09-26 15:58 ` Matthew Fioravante
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5058B07C.1070404@jhuapl.edu \
--to=matthew.fioravante@jhuapl.edu \
--cc=Ian.Campbell@citrix.com \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.