From 47ab5a5132ae54ef69de024cc7ca5890bfddd4fe Mon Sep 17 00:00:00 2001 From: Louis Stermole Date: Wed, 20 Jul 2016 13:30:56 -0500 Subject: [PATCH] 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 Reviewed-by: STEPHEN GLANCY Reviewed-by: JACOB L. HARVEY Tested-by: Jenkins Server Reviewed-by: Brian R. Silver Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36307 Tested-by: Jenkins OP Build CI Tested-by: FSP CI Jenkins Reviewed-by: Daniel M. Crowell --- src/import/chips/p9/procedures/hwp/memory/lib/ecc/ecc.H | 7 ++++--- 1 file 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 8829eab2dce..fa3f405422c 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& i_target, const uint64_t i_galois ) { fapi2::buffer 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& i_target, const mss::mcbist::address i_address ) { fapi2::buffer 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& i_ std::vector& o_error_counts ) { fapi2::buffer 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& 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); }