diff options
| author | Jaymes Wilks <mjwilks@us.ibm.com> | 2017-03-28 11:52:52 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-05-10 15:25:50 -0400 |
| commit | b947a1a4119b06ee50287854e5ea974c0926226e (patch) | |
| tree | 3087d2fa1f5a40354ba9030a176eb969652abd1d /src/usr/secureboot/base/service.C | |
| parent | 17969937b73f10ffe25ba6b4ac22e5d64a2e1f4c (diff) | |
| download | blackbird-hostboot-b947a1a4119b06ee50287854e5ea974c0926226e.tar.gz blackbird-hostboot-b947a1a4119b06ee50287854e5ea974c0926226e.zip | |
Enforce synchronized processor security state
Ensure the secure state is consistent across all processors.
Change-Id: I671253b99d5d87509909681a2cec2380ca6918b0
RTC:167775
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38764
Reviewed-by: Nicholas E. Bofferding <bofferdn@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: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/secureboot/base/service.C')
| -rw-r--r-- | src/usr/secureboot/base/service.C | 31 |
1 files changed, 13 insertions, 18 deletions
diff --git a/src/usr/secureboot/base/service.C b/src/usr/secureboot/base/service.C index 673ec709e..7513f7dc6 100644 --- a/src/usr/secureboot/base/service.C +++ b/src/usr/secureboot/base/service.C @@ -85,20 +85,6 @@ struct SecureRegisterValues */ errlHndl_t getAllSecurityRegisters(std::vector<SecureRegisterValues> & o_regs); -/** - * @brief Adds the values of the Security Registers of the processors in the - * system to an existing error log - * - * @param[in/out] io_err Error Log that the values of the security registers - * will be added to - * NOTE: The state of the system/processors (ie, SCOM - * vs FSI) determines which registers can be included - * - * @return N/A - */ -void addSecurityRegistersToErrlog(errlHndl_t & io_err); - - void* initializeBase(void* unused) { errlHndl_t l_errl = NULL; @@ -140,14 +126,23 @@ bool enabled() } #endif -errlHndl_t getSecuritySwitch(uint64_t& o_regValue, TARGETING::Target* i_targ) +errlHndl_t getSecuritySwitch(uint64_t& o_regValue, TARGETING::Target* i_pProc) +{ + return Singleton<Settings>::instance().getSecuritySwitch(o_regValue, + i_pProc); +} + +errlHndl_t getProcCbsControlRegister(uint64_t& o_regValue, + TARGETING::Target* i_pProc) { - return Singleton<Settings>::instance().getSecuritySwitch(o_regValue,i_targ); + return Singleton<Settings>::instance().getProcCbsControlRegister(o_regValue, + i_pProc); } -errlHndl_t getJumperState(SecureJumperState& o_state, TARGETING::Target* i_targ) +errlHndl_t getJumperState(SecureJumperState& o_state, + TARGETING::Target* i_pProc) { - return Singleton<Settings>::instance().getJumperState(o_state, i_targ); + return Singleton<Settings>::instance().getJumperState(o_state, i_pProc); } errlHndl_t clearSecuritySwitchBits( |
