Skip to content

Commit

Permalink
Don't unmask MCWAT parity errors in Axone
Browse files Browse the repository at this point in the history
Change-Id: Ib419a32447627c0e033c2228c4ec53ddd40754a8
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/96196
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: Marc Gollub <gollub@us.ibm.com>
Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/96394
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
  • Loading branch information
adamant1 authored and dcrowell77 committed May 11, 2020
1 parent 89c2449 commit 8969e01
Showing 1 changed file with 11 additions and 13 deletions.
24 changes: 11 additions & 13 deletions src/import/chips/p9/procedures/hwp/nest/p9_mss_setup_bars.C
Expand Up @@ -2443,7 +2443,7 @@ fapi_try_exit:
/// @return FAPI2_RC_SUCCESS if success, else error code.
///
template<fapi2::TargetType T>
fapi2::ReturnCode unmaskMCFIR(const fapi2::Target<T> i_target)
fapi2::ReturnCode unmaskMCFIR(const fapi2::Target<T> i_target, bool isAxone)
{
FAPI_DBG("Entering");

Expand All @@ -2454,16 +2454,17 @@ fapi2::ReturnCode unmaskMCFIR(const fapi2::Target<T> i_target)
l_mcfiraction.setBit<MCS_MCFIR_MC_INTERNAL_RECOVERABLE_ERROR>();
l_mcfiraction.setBit<MCS_MCFIR_COMMAND_LIST_TIMEOUT>();

if (T == fapi2::TARGET_TYPE_MI)
{
l_mcfiraction.setBit<MCS_MCFIR_MS_WAT_DEBUG_CONFIG_REG_ERROR>();
}

// Setup FIR bits in MC Fault Isolation Mask Register buffer
l_mcfirmask_and.flush<1>();

if (T == fapi2::TARGET_TYPE_MI)
{
if(!isAxone)
{
l_mcfiraction.setBit<MCS_MCFIR_MS_WAT_DEBUG_CONFIG_REG_ERROR>();
l_mcfirmask_and.clearBit<MCS_MCFIR_MS_WAT_DEBUG_CONFIG_REG_ERROR>();
}

l_mcfirmask_and.clearBit<MCS_MCFIR_INBAND_BAR_HIT_WITH_INCORRECT_TTYPE>();
}

Expand All @@ -2479,11 +2480,6 @@ fapi2::ReturnCode unmaskMCFIR(const fapi2::Target<T> i_target)

l_mcfirmask_and.clearBit<MCS_MCFIR_COMMAND_LIST_TIMEOUT>();

if (T == fapi2::TARGET_TYPE_MI)
{
l_mcfirmask_and.clearBit<MCS_MCFIR_MS_WAT_DEBUG_CONFIG_REG_ERROR>();
}

// Defect HW451708, HW451711
// Leave MCS_MCFIR_INVALID_SMF_ACCESS masked if MCD cl_probes are enabled

Expand Down Expand Up @@ -2516,11 +2512,12 @@ fapi2::ReturnCode unmaskMCFIR(
const std::vector<std::pair<fapi2::Target<T>, mcBarData_t>>& i_mcBarDataPair)
{
FAPI_DBG("Entering");
bool isAxone = false;

for (auto l_pair : i_mcBarDataPair)
{
fapi2::Target<T> l_target = l_pair.first;
FAPI_TRY(unmaskMCFIR(l_target));
FAPI_TRY(unmaskMCFIR(l_target, isAxone));

} // Data pair loop

Expand Down Expand Up @@ -2610,6 +2607,7 @@ fapi2::ReturnCode p9_mss_setup_bars(
// Setup BAR for Axone
else if (l_mccChiplets.size() > 0)
{
bool isAxone = true;
// Validate group data from attributes
FAPI_TRY(validateGroupData(l_mccChiplets, true, l_groupData),
"validateGroupData() returns error, l_rc 0x%.8X",
Expand All @@ -2623,7 +2621,7 @@ fapi2::ReturnCode p9_mss_setup_bars(
// Unmask MC FIRs
for (auto l_target : l_miChiplets)
{
FAPI_TRY(unmaskMCFIR(l_target),
FAPI_TRY(unmaskMCFIR(l_target, isAxone),
"unmaskMCFIR() returns error, l_rc 0x%.8X",
uint64_t(fapi2::current_err));
}
Expand Down

0 comments on commit 8969e01

Please sign in to comment.