Re: [SeaBIOS] [PATCH 2/8] tpm: No need to check the return status of measurements

[Kevin O'Connor]

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
> > @@ -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