diff options
author | Mike Baiocchi <mbaiocch@us.ibm.com> | 2018-02-27 17:19:35 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-03-19 11:40:49 -0400 |
commit | 234ef44536ae27e0d83fdbade72d284dd3f1160f (patch) | |
tree | 62735e477ec08690dc5d60558b64ed940f29861c /src/usr/i2c | |
parent | 54d16a1476feb574bc072fd4642245976918760e (diff) | |
download | talos-hostboot-234ef44536ae27e0d83fdbade72d284dd3f1160f.tar.gz talos-hostboot-234ef44536ae27e0d83fdbade72d284dd3f1160f.zip |
Add FFDC to 'No Functional TPM' Fails
When no functional TPMs are detected, but TPMs are required, this
commit will capture the Security Registers of each processor in the
system and add them to the error log. It also updates how the
tpmMarkFailed() function links and commits various error logs.
Change-Id: I2e95bbfcb6ab3f3dff26149f234c219d4280e1fb
CQ:SW417814
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54808
CI-Ready: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/i2c')
-rwxr-xr-x | src/usr/i2c/tpmdd.C | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/usr/i2c/tpmdd.C b/src/usr/i2c/tpmdd.C index a64b43154..11e60b50c 100755 --- a/src/usr/i2c/tpmdd.C +++ b/src/usr/i2c/tpmdd.C @@ -48,6 +48,7 @@ #include <i2c/i2creasoncodes.H> #include <i2c/tpmddreasoncodes.H> #include <i2c/i2cif.H> +#include <secureboot/service.H> #include "tpmdd.H" #include "errlud_i2c.H" @@ -297,12 +298,17 @@ errlHndl_t tpmPerformOp( DeviceFW::OperationType i_opType, } while( 0 ); + if ( err != nullptr ) + { + // Add Security Registers to the error log + SECUREBOOT::addSecurityRegistersToErrlog(err); + } + if( unlock ) { mutex_unlock( & g_tpmMutex ); } - TRACDCOMP( g_trac_tpmdd, EXIT_MRK"tpmPerformOp() - %s", ((NULL == err) ? "No Error" : "With Error") ); |