diff options
author | Nick Bofferding <bofferdn@us.ibm.com> | 2018-03-14 16:50:24 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-05-07 16:23:42 -0400 |
commit | d2c360c9db83212d73b03f744635c5e8f611f0f9 (patch) | |
tree | 5da2252f5b9a57d82aba0ab6e0c2bf85bf543c75 /src/usr/isteps/istep16 | |
parent | b783b0264c7f8c975f2a84218ba79a3ce3b458b1 (diff) | |
download | talos-hostboot-d2c360c9db83212d73b03f744635c5e8f611f0f9.tar.gz talos-hostboot-d2c360c9db83212d73b03f744635c5e8f611f0f9.zip |
Secure Boot: Do not disable external cMFSI fencing in "host IPL complete" istep
Removes extraneous logic in the "host IPL complete" step which used to re-enable
external access to the Centaurs over the cMFSI path. The design point is never
to disable this type of access in the first place, so the noted logic was no
longer necessary.
Change-Id: Id46094cbd6a557a06fab5381448b7d87f12fb4a7
RTC: 187288
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/55890
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>
Reviewed-by: Michael Baiocchi <mbaiocch@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/isteps/istep16')
-rw-r--r-- | src/usr/isteps/istep16/call_host_ipl_complete.C | 31 |
1 files changed, 0 insertions, 31 deletions
diff --git a/src/usr/isteps/istep16/call_host_ipl_complete.C b/src/usr/isteps/istep16/call_host_ipl_complete.C index 1ab0c6760..3993fda8b 100644 --- a/src/usr/isteps/istep16/call_host_ipl_complete.C +++ b/src/usr/isteps/istep16/call_host_ipl_complete.C @@ -39,7 +39,6 @@ #include <fapi2/plat_hwp_invoker.H> #include <p9_switch_rec_attn.H> -#include <p9_switch_cfsim.H> #include <targeting/attrrp.H> #include <sys/internode.h> @@ -274,36 +273,6 @@ void* call_host_ipl_complete (void *io_pArgs) "SUCCESS: p9_switch_rec_attn HWP( ) on target HUID %.8X", TARGETING::get_huid(l_procChip) ); } - - TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, - "Running proc_switch_cfsim HWP on target HUID %.8X", - TARGETING::get_huid(l_procChip) ); - - FAPI_INVOKE_HWP(l_err, p9_switch_cfsim, l_fapiProcTarget); - - if (l_err) - { - TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, - "ERROR 0x%.8X: proc_switch_cfsim HWP returned error", - l_err->reasonCode()); - - // capture the target data in the elog - ErrlUserDetailsTarget(l_procChip).addToLog( l_err ); - - //Create IStep error log and cross reference error that occurred - l_stepError.addErrorDetails( l_err ); - - - //break to end because if proc_switch_cfsim fails - //then FSP does not have FSI control again and system is toast - break; - } - else - { - TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, - "SUCCESS: proc_switch_cfsim HWP( ) on target HUID %.8X", - TARGETING::get_huid(l_procChip) ); - } } //if an error occurred during for loop, break to error handling |