diff options
| author | Dan Crowell <dcrowell@us.ibm.com> | 2018-07-23 22:08:33 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-08-01 16:14:35 -0500 |
| commit | e37d51a556e6748743bc5a83e60c5c8ded22aad8 (patch) | |
| tree | acf0c42536457fff47419791b6576388635f3f4f /src/import/chips/centaur/procedures | |
| parent | 98a657059a5cee86695434ddcf2f6f1f0a774d90 (diff) | |
| download | talos-hostboot-e37d51a556e6748743bc5a83e60c5c8ded22aad8.tar.gz talos-hostboot-e37d51a556e6748743bc5a83e60c5c8ded22aad8.zip | |
Fix array overrun in draminit training advanced
Fixed a counter that wasn't getting reset during each iteration
of the ports. This lead to corruption of local variables.
Change-Id: I7fb31c65d678ef070869a55199b5038f08074e28
CQ: SW439089
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63203
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63249
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import/chips/centaur/procedures')
| -rw-r--r-- | src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C b/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C index bfcd583c5..44fb2f411 100644 --- a/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C +++ b/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C @@ -1826,6 +1826,8 @@ extern "C" FAPI_TRY(dimmGetBadDqBitmap(i_target_mba, l_port, l_dimm, l_rnk, l_dqBitmap)); + l_index_sp = 0; // reset for the next time through the loop + for (l_index0 = 0; l_index0 < DIMM_DQ_RANK_BITMAP_SIZE; l_index0++) { if (l_index0 < DATA_BYTES_PER_PORT) |
