diff options
author | Zane Shelley <zshelle@us.ibm.com> | 2017-08-31 16:47:10 -0500 |
---|---|---|
committer | Zane C. Shelley <zshelle@us.ibm.com> | 2017-09-01 16:18:06 -0400 |
commit | 8f4664630fef9177bf09d891152f668ee8a53157 (patch) | |
tree | 79564ae329c4fc2b7a7432dbedce84a20aece60c /src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C | |
parent | 82ce429fe52ff6a3c51e5dde02883675075a3fab (diff) | |
download | talos-hostboot-8f4664630fef9177bf09d891152f668ee8a53157.tar.gz talos-hostboot-8f4664630fef9177bf09d891152f668ee8a53157.zip |
PRD: remove assert for debug purposes
Change-Id: I2db9d5868e7f38f06719c0c47850032bd10ef298
CQ: SW400798
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/45531
Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com>
Reviewed-by: Brian J. Stegmiller <bjs@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>
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: Zane C. Shelley <zshelle@us.ibm.com>
Diffstat (limited to 'src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C')
-rw-r--r-- | src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C | 25 |
1 files changed, 19 insertions, 6 deletions
diff --git a/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C b/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C index 4dc2dc7e3..d37793455 100644 --- a/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C +++ b/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C @@ -67,7 +67,17 @@ uint32_t MemTdCtlr<T>::handleCmdComplete( STEP_CODE_DATA_STRUCT & io_sc ) #else // IPL only - PRDF_ASSERT( isInMdiaMode() ); // MDIA must be running. + // TODO: RTC 179251 asserting here doesn't give us enough FFDC to debug + // why we got this erroneous attention. Eventually, we will want + // to add the capture data to the assert error log. Until then + // exit with a bad RC and make the error log predictive. + // PRDF_ASSERT( isInMdiaMode() ); // MDIA must be running. + if ( !isInMdiaMode() ) + { + PRDF_ERR( PRDF_FUNC "IPL cmd complete attn outside of MDIA" ); + o_rc = FAIL; + break; + } // Inform MDIA the command has completed and PRD is starting analysis. o_rc = mdiaSendEventMsg( iv_chip->getTrgt(), MDIA::RESET_TIMER ); @@ -137,11 +147,14 @@ uint32_t MemTdCtlr<T>::handleCmdComplete( STEP_CODE_DATA_STRUCT & io_sc ) #ifndef __HOSTBOOT_RUNTIME // IPL only - // Tell MDIA to skip further analysis on this target. - uint32_t l_rc = mdiaSendEventMsg( iv_chip->getTrgt(), - MDIA::STOP_TESTING ); - if ( SUCCESS != l_rc ) - PRDF_ERR( PRDF_FUNC "mdiaSendEventMsg(STOP_TESTING) failed" ); + if ( isInMdiaMode() ) + { + // Tell MDIA to skip further analysis on this target. + uint32_t l_rc = mdiaSendEventMsg( iv_chip->getTrgt(), + MDIA::STOP_TESTING ); + if ( SUCCESS != l_rc ) + PRDF_ERR( PRDF_FUNC "mdiaSendEventMsg(STOP_TESTING) failed" ); + } #endif } |