diff options
author | Louis Stermole <stermole@us.ibm.com> | 2016-07-20 13:30:56 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-02-11 21:41:26 -0500 |
commit | 47ab5a5132ae54ef69de024cc7ca5890bfddd4fe (patch) | |
tree | 78a9f7be53677285cbe4f7b14a550520aa946e51 | |
parent | 506ba5543abc4d2928c084181287c919d057cb51 (diff) | |
download | talos-hostboot-47ab5a5132ae54ef69de024cc7ca5890bfddd4fe.tar.gz talos-hostboot-47ab5a5132ae54ef69de024cc7ca5890bfddd4fe.zip |
Fixing fail in mss_ecc_api_ut.C (p9_mss_ut.exe) unit test
Change-Id: Ibc1624b6fab7411f403fd1282630187e266e12b5
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/27282
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: JACOB L. HARVEY <jlharvey@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Brian R. Silver <bsilver@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36307
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
-rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/lib/ecc/ecc.H | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/ecc/ecc.H b/src/import/chips/p9/procedures/hwp/memory/lib/ecc/ecc.H index 8829eab2d..fa3f40542 100644 --- a/src/import/chips/p9/procedures/hwp/memory/lib/ecc/ecc.H +++ b/src/import/chips/p9/procedures/hwp/memory/lib/ecc/ecc.H @@ -99,7 +99,7 @@ inline fapi2::ReturnCode set_hwms( const fapi2::Target<T>& i_target, const uint64_t i_galois ) { fapi2::buffer<uint64_t> l_buffer; - uint8_t l_symbol; + uint8_t l_symbol = 0; // galois value of 0 means to clear the mark so only fill in fields if non-zero if (i_galois != 0) @@ -168,7 +168,7 @@ inline fapi2::ReturnCode set_fwms( const fapi2::Target<T>& i_target, const mss::mcbist::address i_address ) { fapi2::buffer<uint64_t> l_buffer = 0; - uint8_t l_symbol; + uint8_t l_symbol = 0; // galois value of 0 means to clear the mark so only fill in fields if non-zero if (i_galois != 0) @@ -449,7 +449,7 @@ inline fapi2::ReturnCode get_per_symbol_error_counts( const fapi2::Target<T>& i_ std::vector<uint64_t>& o_error_counts ) { fapi2::buffer<uint64_t> l_buffer; - uint64_t l_count; + uint64_t l_count = 0; for (uint64_t l_index = 0; l_index < TT::NUM_MBSSYM_REGS; ++l_index) { @@ -457,6 +457,7 @@ inline fapi2::ReturnCode get_per_symbol_error_counts( const fapi2::Target<T>& i_ for (uint64_t l_symbol = 0; l_symbol < TT::MODAL_SYMBOL_COUNTERS_PER_REG; ++l_symbol) { + l_count = 0; mss::ecc::modal_symbol_count::get_count(l_buffer, l_symbol, l_count); o_error_counts.push_back(l_count); } |