summaryrefslogtreecommitdiffstats
path: root/src/usr/diag/prdf/plat
diff options
context:
space:
mode:
authorZane Shelley <zshelle@us.ibm.com>2016-12-02 16:19:05 -0600
committerZane C. Shelley <zshelle@us.ibm.com>2016-12-09 18:17:37 -0500
commit7791bc24b3d83bcb14572aaba69d8d32f0c917b1 (patch)
tree696c99246317b2961a443b02f5a7a523ec5ce631 /src/usr/diag/prdf/plat
parent074d8aa2d032263ca5d65f187e59b01223bde97b (diff)
downloadtalos-hostboot-7791bc24b3d83bcb14572aaba69d8d32f0c917b1.tar.gz
talos-hostboot-7791bc24b3d83bcb14572aaba69d8d32f0c917b1.zip
PRD: add default capture data support for ECC attentions
Change-Id: I87a8b7afc399633f42fbccf53896a6a10a7d327c RTC: 165374 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33357 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33685 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Diffstat (limited to 'src/usr/diag/prdf/plat')
-rw-r--r--src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C8
-rw-r--r--src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C2
2 files changed, 9 insertions, 1 deletions
diff --git a/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C b/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C
index 6823e8254..156c6d99a 100644
--- a/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C
+++ b/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C
@@ -26,6 +26,7 @@
#include <prdfMemTdCtlr.H>
#include <prdfMemAddress.H>
+#include <prdfMemCaptureData.H>
#include <prdfP9McbistExtraSig.H>
using namespace TARGETING;
@@ -110,6 +111,13 @@ uint32_t MemTdCtlr<T>::handleCmdComplete( STEP_CODE_DATA_STRUCT & io_sc )
} while (0);
+ // Gather capture data even if something failed above.
+ // NOTE: There is no need to capture the data if the command completed
+ // successfully with no errors because the error log will not be
+ // committed.
+ if ( !io_sc.service_data->queryDontCommitErrl() )
+ MemCaptureData::addEccData<T>( iv_chip, io_sc );
+
if ( SUCCESS != o_rc )
{
PRDF_ERR( PRDF_FUNC "Failed on 0x%08x", iv_chip->getHuid() );
diff --git a/src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C b/src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C
index b5ab1aaa5..81a03f635 100644
--- a/src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C
+++ b/src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C
@@ -81,7 +81,7 @@ void commitErrl( errlHndl_t i_errl, TargetHandle_t i_mba )
if ( NULL != i_errl )
{
// Add capture data
- CenMbaCaptureData::addMemEccData( i_mba, i_errl );
+ CenMbaCaptureData::addEccData( i_mba, i_errl );
// Add traces
i_errl->collectTrace( PRDF_COMP_NAME, 512 );
OpenPOWER on IntegriCloud