[SeaBIOS] [PATCH 2/8] tpm: No need to check the return status of measurements
Kevin O'Connor
kevin at koconnor.net
Thu Dec 31 01:17:44 CET 2015
On Wed, Dec 30, 2015 at 07:09:54PM -0500, Stefan Berger wrote:
> On 12/30/2015 02:31 PM, Kevin O'Connor wrote:
> >The low-level measurement functions already handle error conditions,
> >there is no need to check for the errors in the high level measurement
> >functions.
> >
> >Signed-off-by: Kevin O'Connor <kevin at koconnor.net>
> >---
>
> >@@ -507,7 +478,12 @@ tpm_setup(void)
> > if (runningOnXen())
> > return;
>
> This looks like a for Xen where we will be missing too much. Cc'in Xu Quan
> for this. I think Xen likely only wants to skip the TPM_Startup in
> tpm_startup() but not the retrieval of the durations and timeouts and so on.
> This part may need to move.
That makes sense, but note that the position of the check hasn't
changed in this series.
I wonder if the error from TPM_Startup on Xen could just be ignored as
is done for CONFIG_COREBOOT.
-Kevin
More information about the SeaBIOS
mailing list