summaryrefslogtreecommitdiffstats
path: root/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C
diff options
context:
space:
mode:
authorStephen Glancy <sglancy@us.ibm.com>2017-07-05 10:07:44 -0500
committerDean Sanner <dsanner@us.ibm.com>2017-07-11 12:19:10 -0400
commitf3dcd540d159a6e919146afeab817aad95efa50b (patch)
treeb09d8d947f8927618de836a4c740e981d1d8e416 /src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C
parenta92b4603d7a0e642c0e8801e264434b41a9d1426 (diff)
downloadtalos-hostboot-f3dcd540d159a6e919146afeab817aad95efa50b.tar.gz
talos-hostboot-f3dcd540d159a6e919146afeab817aad95efa50b.zip
Fixed memdiags fail
Change-Id: I5976833f28a34c2083ad475af065bf372129e7c0 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42756 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt K. Light <mklight@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42775 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: Dean Sanner <dsanner@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C')
-rw-r--r--src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C5
1 files changed, 3 insertions, 2 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C b/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C
index df159cc86..5f182a8b9 100644
--- a/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C
+++ b/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C
@@ -83,9 +83,10 @@ extern "C"
// assert if we have exceeded the allowed repairs
for (const auto& l_dimm : mss::find_targets<fapi2::TARGET_TYPE_DIMM>(l_mca))
{
+ // Note: using MCA here as the scoms used to collect FFDC data fail on the DIMM level target
FAPI_ASSERT( !(l_repairs_exceeded.getBit(mss::index(l_dimm))),
- fapi2::MSS_MEMDIAGS_REPAIRS_EXCEEDED().set_TARGET(l_dimm),
- "p9_mss_memdiag bad bit repairs exceeded %s", mss::c_str(l_dimm) );
+ fapi2::MSS_MEMDIAGS_REPAIRS_EXCEEDED().set_TARGET(l_mca),
+ "p9_mss_memdiag bad bit repairs exceeded %s", mss::c_str(l_mca) );
}
#ifdef __HOSTBOOT_MODULE
OpenPOWER on IntegriCloud