diff options
author | Stephen Glancy <sglancy@us.ibm.com> | 2017-07-05 10:07:44 -0500 |
---|---|---|
committer | Dean Sanner <dsanner@us.ibm.com> | 2017-07-11 12:19:10 -0400 |
commit | f3dcd540d159a6e919146afeab817aad95efa50b (patch) | |
tree | b09d8d947f8927618de836a4c740e981d1d8e416 /src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C | |
parent | a92b4603d7a0e642c0e8801e264434b41a9d1426 (diff) | |
download | talos-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.C | 5 |
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 |