diff options
author | Bill Schwartz <whs@us.ibm.com> | 2013-10-17 08:03:36 -0500 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2013-11-08 15:19:40 -0600 |
commit | 06fdca58e18f8a6dae366a12e03b97c7b28d5742 (patch) | |
tree | fdb22b6969b05b85cb7b097455ed4101374a5ac1 /src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C | |
parent | fd4fe707fca2059867b4573f3f6f1bd3f708b542 (diff) | |
download | talos-hostboot-06fdca58e18f8a6dae366a12e03b97c7b28d5742.tar.gz talos-hostboot-06fdca58e18f8a6dae366a12e03b97c7b28d5742.zip |
istepreasoncodes.H is a merge bottleneck
Eliminate congestion on istepreasoncodes.H by removing error tags and
moduleId/Reason codes from iSteps calling IStepError.addErrorDetails.
ISteps pass only the failing error log. IStepError.addErrorDetails will get
the iStep and substep from the istep dispatcher and add to user data.
Change-Id: If8f8796bfeafb80b5dbcea21629137260b07bc72
RTC: 59658
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/6719
Tested-by: Jenkins Server
Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com>
Reviewed-by: Brian H. Horton <brianh@linux.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C')
-rw-r--r-- | src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C | 44 |
1 files changed, 10 insertions, 34 deletions
diff --git a/src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C b/src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C index 7f8a0e8eb..1217c53dd 100644 --- a/src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C +++ b/src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C @@ -110,7 +110,8 @@ void* call_sbe_centaur_init( void *io_pArgs ) uint8_t cur_ec = (*l_membuf_iter)->getAttr<TARGETING::ATTR_EC>(); - TRACDCOMP( ISTEPS_TRACE::g_trac_isteps_trace, INFO_MRK"call_sbe_centaur_init() - Find SBE image in PNOR"); + TRACDCOMP( ISTEPS_TRACE::g_trac_isteps_trace, INFO_MRK, + "call_sbe_centaur_init() - Find SBE image in PNOR"); l_errl = SBE::findSBEInPnor(l_membuf_target, l_sbePnorAddr, @@ -118,7 +119,8 @@ void* call_sbe_centaur_init( void *io_pArgs ) NULL); if (l_errl) { - TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, ERR_MRK"call_sbe_centaur_init() - Error getting image from PNOR. ec=0x%.2X", + TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, ERR_MRK, + "call_sbe_centaur_init() - Error getting image from PNOR. ec=0x%.2X", cur_ec ); TRACFBIN(ISTEPS_TRACE::g_trac_isteps_trace, @@ -127,23 +129,10 @@ void* call_sbe_centaur_init( void *io_pArgs ) // capture the target data in the elog ErrlUserDetailsTarget(l_membuf_target).addToLog( l_errl ); - /*@ - * @errortype - * @reasoncode ISTEP_SBE_PNOR_CENTAUR_FAILED - * @severity ERRORLOG::ERRL_SEV_UNRECOVERABLE - * @moduleid ISTEP_SBE_CENTAUR_INIT - * @userdata1 bytes 0-1: plid identifying first error - * bytes 2-3: reason code of first error - * @userdata2 bytes 0-1: total number of elogs included - * bytes 2-3: N/A - * @devdesc call to find the centaur sbe image in pnor - * in the sbe_centaur_init function to initialize - * the centuars has failed - */ - l_StepError.addErrorDetails(ISTEP_SBE_PNOR_CENTAUR_FAILED, - ISTEP_SBE_CENTAUR_INIT, - l_errl); + // Create IStep error log and cross reference error that occurred + l_StepError.addErrorDetails( l_errl ); + // Commit Error errlCommit( l_errl, HWPF_COMP_ID ); break; } @@ -299,22 +288,10 @@ void* call_sbe_centaur_init( void *io_pArgs ) // capture the target data in the elog ErrlUserDetailsTarget(l_membuf_target).addToLog( l_errl ); - /*@ - * @errortype - * @reasoncode ISTEP_SBE_CENTAUR_INIT_FAILED - * @severity ERRORLOG::ERRL_SEV_UNRECOVERABLE - * @moduleid ISTEP_SBE_CENTAUR_INIT - * @userdata1 bytes 0-1: plid identifying first error - * bytes 2-3: reason code of first error - * @userdata2 bytes 0-1: total number of elogs included - * bytes 2-3: N/A - * @devdesc call to proc_swl_build to build the sleep - * winkle image has failed - */ - l_StepError.addErrorDetails(ISTEP_SBE_CENTAUR_INIT_FAILED, - ISTEP_SBE_CENTAUR_INIT, - l_errl); + // Create IStep error log and cross reference error that occurred + l_StepError.addErrorDetails( l_errl ); + // Commit Error errlCommit( l_errl, HWPF_COMP_ID ); break; // break out of memBuf loop @@ -328,7 +305,6 @@ void* call_sbe_centaur_init( void *io_pArgs ) } // end for - TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, "call_sbe_centaur_init exit" ); |