summaryrefslogtreecommitdiffstats
path: root/src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C
diff options
context:
space:
mode:
authorBill Schwartz <whs@us.ibm.com>2013-10-17 08:03:36 -0500
committerA. Patrick Williams III <iawillia@us.ibm.com>2013-11-08 15:19:40 -0600
commit06fdca58e18f8a6dae366a12e03b97c7b28d5742 (patch)
treefdb22b6969b05b85cb7b097455ed4101374a5ac1 /src/usr/hwpf/hwp/sbe_centaur_init/sbe_centaur_init.C
parentfd4fe707fca2059867b4573f3f6f1bd3f708b542 (diff)
downloadtalos-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.C44
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" );
OpenPOWER on IntegriCloud