summaryrefslogtreecommitdiffstats
path: root/src/usr/sbeio
diff options
context:
space:
mode:
authorChristian Geddes <crgeddes@us.ibm.com>2018-02-14 14:19:28 -0600
committerDaniel M. Crowell <dcrowell@us.ibm.com>2018-02-21 17:59:20 -0500
commit2c4b416ae0cfaf7b77954d5d2d5d589427751383 (patch)
tree72804d6392e677da1ca06af31244aea054368136 /src/usr/sbeio
parent4a60925ef57e65a4af51703350b75383d9b3c674 (diff)
downloadtalos-hostboot-2c4b416ae0cfaf7b77954d5d2d5d589427751383.tar.gz
talos-hostboot-2c4b416ae0cfaf7b77954d5d2d5d589427751383.zip
Remove if that was catching SBE chipop err logs and forcing reboot
Before Hostboot had the ability to parse SBE FFDC, we would perform a special shutdown that would request the FSP to parse the SBE FFDC in the event of a SBE chip-op (PSU or FIFO) failure. Now that hostboot can do this we do not need to go down this path in this case. The only time we should do this is if the SbeRetryHandler has decided that this is required. CQ: SW417255 Change-Id: I8f52d52294d84c9e10faf7d8bc500808463aa57f Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54063 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> Reviewed-by: Nicholas E. Bofferding <bofferdn@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/sbeio')
-rw-r--r--src/usr/sbeio/sbe_fifodd.C30
-rw-r--r--src/usr/sbeio/sbe_psudd.C37
2 files changed, 0 insertions, 67 deletions
diff --git a/src/usr/sbeio/sbe_fifodd.C b/src/usr/sbeio/sbe_fifodd.C
index 50dbd880d..5906452ec 100644
--- a/src/usr/sbeio/sbe_fifodd.C
+++ b/src/usr/sbeio/sbe_fifodd.C
@@ -41,7 +41,6 @@
#include "sbe_fifodd.H"
#include <sbeio/sbe_ffdc_package_parser.H>
#include <sbeio/sbe_ffdc_parser.H>
-#include <initservice/initserviceif.H>
#include <kernel/pagemgr.H>
#include <fapi2.H>
#include <set_sbe_error.H>
@@ -133,35 +132,6 @@ errlHndl_t SbeFifo::performFifoChipOp(TARGETING::Target * i_target,
mutex_unlock(&l_fifoOpMux);
- if( errl && (SBEIO_FIFO == errl->moduleId()) )
- {
- SBE_TRACF( "Forcing shutdown for FSP to collect FFDC" );
-
- //commit the original error after pulling some data out
- uint32_t orig_plid = errl->plid();
- uint32_t orig_rc = errl->reasonCode();
- uint32_t orig_mod = errl->moduleId();
- ERRORLOG::errlCommit( errl, SBEIO_COMP_ID );
- /*@
- * @errortype
- * @moduleid SBEIO_FIFO
- * @reasoncode SBEIO_HWSV_COLLECT_SBE_RC
- * @userdata1 PLID of original error log
- * @userdata2[00:31] Original RC
- * @userdata2[32:63] Original Module Id
- *
- * @devdesc SBE error, force HWSV to collect FFDC
- * @custdesc Firmware error communicating with boot device
- */
- errl = new ErrlEntry(ERRL_SEV_UNRECOVERABLE,
- SBEIO_FIFO,
- SBEIO_HWSV_COLLECT_SBE_RC,
- orig_plid,
- TWO_UINT32_TO_UINT64(orig_rc,orig_mod));
- INITSERVICE::doShutdownWithError( SBEIO_HWSV_COLLECT_SBE_RC,
- TARGETING::get_huid(i_target) );
- }
-
SBE_TRACD(EXIT_MRK "performFifoChipOp");
return errl;
diff --git a/src/usr/sbeio/sbe_psudd.C b/src/usr/sbeio/sbe_psudd.C
index 58a891e26..77b8f4824 100644
--- a/src/usr/sbeio/sbe_psudd.C
+++ b/src/usr/sbeio/sbe_psudd.C
@@ -37,7 +37,6 @@
#include <targeting/common/target.H>
#include <errl/errlreasoncodes.H>
#include <sbeio/sbeioreasoncodes.H>
-#include <initservice/initserviceif.H> //@todo-RTC:149454-Remove
#include <sbeio/sbe_ffdc_package_parser.H>
#include <sbeio/sbe_psudd.H>
#include <sbeio/sbe_ffdc_parser.H>
@@ -165,42 +164,6 @@ errlHndl_t SbePsu::performPsuChipOp(TARGETING::Target * i_target,
mutex_unlock(&l_psuOpMux);
- if( errl && (SBEIO_PSU == errl->moduleId())
- // For this special case pass back errl without commiting or
- // collecting FFDC/shutting down
- && (SBE_PSU_SET_UNSECURE_MEMORY_REGION_CMD != i_pPsuRequest->command)
- )
- {
- SBE_TRACF( "Forcing shutdown for FSP to collect FFDC" );
-
- //commit the original error after pulling some data out
- uint32_t orig_plid = errl->plid();
- uint32_t orig_rc = errl->reasonCode();
- uint32_t orig_mod = errl->moduleId();
- ERRORLOG::errlCommit( errl, SBEIO_COMP_ID );
- /*@
- * @errortype
- * @moduleid SBEIO_PSU
- * @reasoncode SBEIO_HWSV_COLLECT_SBE_RC
- * @userdata1 PLID of original error log
- * @userdata2[00:31] Original RC
- * @userdata2[32:63] Original Module Id
- *
- * @devdesc SBE error, force HWSV to collect FFDC
- * @custdesc Firmware error communicating with boot device
- */
- errl = new ErrlEntry(ERRL_SEV_UNRECOVERABLE,
- SBEIO_PSU,
- SBEIO_HWSV_COLLECT_SBE_RC,
- orig_plid,
- TWO_UINT32_TO_UINT64(orig_rc,orig_mod));
- MAGIC_INST_GET_SBE_TRACES(
- i_target->getAttr<TARGETING::ATTR_POSITION>(),
- SBEIO_HWSV_COLLECT_SBE_RC);
- INITSERVICE::doShutdownWithError( SBEIO_HWSV_COLLECT_SBE_RC,
- TARGETING::get_huid(i_target) );
- }
-
SBE_TRACD(EXIT_MRK "performPsuChipOp");
return errl;
OpenPOWER on IntegriCloud