Skip to content

Commit

Permalink
Fixing fail in mss_ecc_api_ut.C (p9_mss_ut.exe) unit test
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
stermole authored and dcrowell77 committed Feb 12, 2017
1 parent 506ba55 commit 47ab5a5
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/import/chips/p9/procedures/hwp/memory/lib/ecc/ecc.H
Expand Up @@ -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)
Expand Down Expand Up @@ -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)
Expand Down Expand Up @@ -449,14 +449,15 @@ 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)
{
FAPI_TRY( mss::ecc::modal_symbol_count::read(i_target, l_index, l_buffer) );

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);
}
Expand Down

0 comments on commit 47ab5a5

Please sign in to comment.