[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough conf
From: |
Paul Brook |
Subject: |
Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option |
Date: |
Tue, 13 Dec 2011 17:25:51 +0000 |
User-agent: |
KMail/1.13.7 (Linux/3.1.0-1-amd64; KDE/4.6.5; x86_64; ; ) |
> On 12/12/2011 11:51 PM, Paul Brook wrote:
> >>>> +tpm_passthrough="no"
> >>>
> >>> Same as before, please probe for existence.
> >>
> >> We would be probing for /dev/tpm0. Is that really what we want that this
> >> driver only gets compiled if /dev/tpm0 is (currently) available?
> >
> > If what you say is true then this code should always be enabled.
>
> Michael Tsirkin previously requested that there be an option for the TPM
> passthrough driver to be selectively enabled since at least using
> /dev/tpm0 may not be what everybody wants. The passthrough driver at
> some point will also be able to use sockets to communicate with a TPM
> when a file descriptor is passed to Qemu, so maybe that changes then?
Surely that's a runtime decision made by the qemu user, not a compile time
decision made by the distribution vendor. Testing functionality of the build
machine is fundamentally wrong. At best it completely breaks cross compiling,
at worst it subtly breaks building on a dev machine and running on a
production server.
Configure time probing only makes sense for code that will fail to build if
external compile time dependencies are not present.
If you really think building qemu without this functionality is useful, then
maybe add a configure option to disable it. With the possible exception of
developer debugging aids, code that is not enabled by default is clearly not
worth having and should be removed altogether.
Paul
- [Qemu-devel] [PATCH V13 3/7] Add a debug register, (continued)
- [Qemu-devel] [PATCH V13 3/7] Add a debug register, Stefan Berger, 2011/12/12
- [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Stefan Berger, 2011/12/12
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Anthony Liguori, 2011/12/12
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Stefan Berger, 2011/12/12
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Paul Brook, 2011/12/12
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Stefan Berger, 2011/12/13
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Michael S. Tsirkin, 2011/12/13
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Anthony Liguori, 2011/12/13
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Stefan Berger, 2011/12/13
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option, Paul Brook, 2011/12/13
- Re: [Qemu-devel] [PATCH V13 6/7] Introduce --enable-tpm-passthrough configure option,
Paul Brook <=
[Qemu-devel] [PATCH V13 7/7] Add fd parameter for TPM passthrough driver, Stefan Berger, 2011/12/12
[Qemu-devel] [PATCH V13 1/7] Support for TPM command line options, Stefan Berger, 2011/12/12
[Qemu-devel] [PATCH V13 2/7] Add TPM (frontend) hardware interface (TPM TIS) to Qemu, Stefan Berger, 2011/12/12
[Qemu-devel] [PATCH V13 5/7] Add a TPM Passthrough backend driver implementation, Stefan Berger, 2011/12/12