From 222af0c322a6e0383000aaf3da130dd878f084c9 Mon Sep 17 00:00:00 2001 From: Benjamin Weisenbeck Date: Thu, 29 Jun 2017 13:12:45 -0500 Subject: [PATCH] PRD: Allow clearing FIR bits on Unit Checkstops There is no planned support for unit checkstops in P9, so we don't have any code in place for properly disabling checkstopped units. It shouldn't matter, but when we get a bit mistakenly configured as a Unit CS, we end up with a flood of errors for the uncleared bit. Change-Id: Ia7104b31c988434ca6df74889afad14c7ecd158b Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42613 Tested-by: Jenkins Server Reviewed-by: Caleb N. Palmer Reviewed-by: Zane C. Shelley Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42635 Tested-by: FSP CI Jenkins Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW --- .../prdf/common/framework/register/prdfResetErrorRegister.C | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/usr/diag/prdf/common/framework/register/prdfResetErrorRegister.C b/src/usr/diag/prdf/common/framework/register/prdfResetErrorRegister.C index cf543da5b17..b1da2c31785 100755 --- a/src/usr/diag/prdf/common/framework/register/prdfResetErrorRegister.C +++ b/src/usr/diag/prdf/common/framework/register/prdfResetErrorRegister.C @@ -109,9 +109,7 @@ ResetAndMaskErrorRegister::Reset(const BitKey & bit_list, { int32_t rc = SUCCESS; // Don't do reset on CS. - if ((CHECK_STOP != error.service_data->getPrimaryAttnType()) && //@pw01 - (UNIT_CS != error.service_data->getPrimaryAttnType()) && - (UNIT_CS != error.service_data->getSecondaryAttnType())) + if (CHECK_STOP != error.service_data->getPrimaryAttnType()) { #ifndef __HOSTBOOT_MODULE ServiceDataCollector & sdc = *(error.service_data);