diff options
| author | Caleb Palmer <cnpalmer@us.ibm.com> | 2018-03-05 10:58:42 -0600 |
|---|---|---|
| committer | Zane C. Shelley <zshelle@us.ibm.com> | 2018-03-06 14:42:36 -0500 |
| commit | 18a73baccdc224201d38f73d5000e1f88f1bd52d (patch) | |
| tree | 04f12300722ae8f53d87977c948e6314248513a3 /src/usr/diag | |
| parent | 25e991e8b35268883c08a9691972f0912e183f4d (diff) | |
| download | talos-hostboot-18a73baccdc224201d38f73d5000e1f88f1bd52d.tar.gz talos-hostboot-18a73baccdc224201d38f73d5000e1f88f1bd52d.zip | |
PRD: Don't skip TPS after failed MemDealloc calls
Change-Id: I0352df92781cc8ea84069fed44422e1f1d69a550
CQ: SW419826
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/55060
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com>
Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com>
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/55123
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>
Diffstat (limited to 'src/usr/diag')
| -rw-r--r-- | src/usr/diag/prdf/common/plat/mem/prdfMemEccAnalysis.C | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/usr/diag/prdf/common/plat/mem/prdfMemEccAnalysis.C b/src/usr/diag/prdf/common/plat/mem/prdfMemEccAnalysis.C index cc5fcb60c..2c97a8202 100644 --- a/src/usr/diag/prdf/common/plat/mem/prdfMemEccAnalysis.C +++ b/src/usr/diag/prdf/common/plat/mem/prdfMemEccAnalysis.C @@ -72,8 +72,7 @@ uint32_t __handleMemUe( ExtensibleChip * i_chip, const MemAddr & i_addr, #ifdef __HOSTBOOT_RUNTIME // Dynamically deallocate the rank. - o_rc = MemDealloc::rank<T>( i_chip, rank ); - if ( SUCCESS != o_rc ) + if ( SUCCESS != MemDealloc::rank<T>( i_chip, rank ) ) { PRDF_ERR( PRDF_FUNC "MemDealloc::rank<T>(0x%08x,m%ds%d) failed", i_chip->getHuid(), rank.getMaster(), rank.getSlave() ); @@ -203,8 +202,7 @@ uint32_t maskMemPort<TYPE_MCA>( ExtensibleChip * i_chip ) #ifdef __HOSTBOOT_RUNTIME // Dynamically deallocate the port. - o_rc = MemDealloc::port<TYPE_MCA>( i_chip ); - if ( SUCCESS != o_rc ) + if ( SUCCESS != MemDealloc::port<TYPE_MCA>( i_chip ) ) { PRDF_ERR( PRDF_FUNC "MemDealloc::port<TYPE_MCA>(0x%08x) failed", i_chip->getHuid() ); @@ -603,8 +601,7 @@ uint32_t handleMemCe( ExtensibleChip * i_chip, const MemAddr & i_addr, if ( i_isHard ) { // Dynamically deallocate the page. - o_rc = MemDealloc::page<T>( i_chip, i_addr ); - if ( SUCCESS != o_rc ) + if ( SUCCESS != MemDealloc::page<T>( i_chip, i_addr ) ) { PRDF_ERR( PRDF_FUNC "MemDealloc::page(0x%08x) failed", i_chip->getHuid() ); |
