diff options
| author | Marty Gloff <mgloff@us.ibm.com> | 2018-03-28 11:00:35 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-04-09 16:38:00 -0400 |
| commit | 7383c3a4fbaff710ecbefb3e9eeeae7320d9f8b1 (patch) | |
| tree | fabf0c55c9f4c048b330ec60ea97e97c5f2ebaea /src/usr/initservice | |
| parent | 519b09db143e87b3d43f39790486f6c5dc1be96b (diff) | |
| download | talos-hostboot-7383c3a4fbaff710ecbefb3e9eeeae7320d9f8b1.tar.gz talos-hostboot-7383c3a4fbaff710ecbefb3e9eeeae7320d9f8b1.zip | |
Get Hostboot Dump after Hostboot Crash
When Hostboot has a TI with a PLID rather than a Reason Code (RC), HWSV does not
attempt the Hostboot Dump. Thought is that happens per the design as the error
log related to the PLID sent by Hostboot should already have the required
information. There are instances where the error log does not have sufficient
data and a dump would be beneficial. A path is being added so an error log can
be flagged as needing a Hostboot dump.
Change-Id: I97972308c70e7210f578fb818563bb9b0cd940b4
CQ: SW420219
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/56600
CI-Ready: Daniel M. Crowell <dcrowell@us.ibm.com>
CI-Ready: Martin Gloff <mgloff@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@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: Corey V. Swenson <cswenson@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/initservice')
| -rw-r--r-- | src/usr/initservice/baseinitsvc/initservice.C | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/usr/initservice/baseinitsvc/initservice.C b/src/usr/initservice/baseinitsvc/initservice.C index 58a86ac26..b16fd213e 100644 --- a/src/usr/initservice/baseinitsvc/initservice.C +++ b/src/usr/initservice/baseinitsvc/initservice.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2011,2017 */ +/* Contributors Listed Below - COPYRIGHT 2011,2018 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -260,11 +260,13 @@ errlHndl_t InitService::startTask( * */ const bool hbSwError = true; + const bool hbDump = true; l_errl = new ERRORLOG::ErrlEntry( ERRORLOG::ERRL_SEV_CRITICAL_SYS_TERM, INITSERVICE::BASE_INITSVC_MOD_ID, INITSERVICE::WAIT_TASK_FAILED, - l_tidretrc, l_childsts, hbSwError); + l_tidretrc, l_childsts, hbSwError, + hbDump); // Add Printk Buffer for FFDC. ERRORLOG::ErrlUserDetailsPrintk().addToLog(l_errl); @@ -392,11 +394,13 @@ errlHndl_t InitService::executeFn( * */ const bool hbSwError = true; + const bool hbDump = true; l_errl = new ERRORLOG::ErrlEntry( ERRORLOG::ERRL_SEV_CRITICAL_SYS_TERM, INITSERVICE::BASE_INITSVC_MOD_ID, INITSERVICE::WAIT_FN_FAILED, - l_tidretrc, l_childsts, hbSwError); + l_tidretrc, l_childsts, hbSwError, + hbDump); // Add Printk Buffer for FFDC. ERRORLOG::ErrlUserDetailsPrintk().addToLog(l_errl); @@ -775,6 +779,7 @@ void InitService::doShutdown(uint64_t i_status, // Ensure no one is manpulating the registry lists and that only one // thread actually executes the shutdown path. mutex_lock(&iv_registryMutex); + if (iv_shutdownInProgress) { // switch the failing status if an RC comes in after |
