Skip to content

Commit

Permalink
PRD: Do not abort on UE during MBA TPS
Browse files Browse the repository at this point in the history
Change-Id: I9f21151fddd0d95a593df486eb419b7e78eb2677
CQ: SW433373
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/60429
Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com>
Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/60564
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
  • Loading branch information
zane131 committed Jun 17, 2018
1 parent 2024675 commit 3476860
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/usr/diag/prdf/plat/mem/prdfMemTps_rt.C
Original file line number Diff line number Diff line change
Expand Up @@ -1277,10 +1277,9 @@ uint32_t TpsEvent<TYPE_MBA>::analyzeEccErrors( const uint32_t & i_eccAttns,
break;
}

// Abort this procedure because additional repairs will likely
// not help (also avoids complication of having UE and MPE at
// the same time).
o_done = true; break;
// Do not abort the procedure. UEs are expected. Since the command
// has to stop on error due to the hardware bugs, we need to resume
// the command and ensure it gets to the end of the rank.
}

// If there was an MPE.
Expand Down

0 comments on commit 3476860

Please sign in to comment.