Skip to content

Commit

Permalink
Fix bugs in core checkstop escalation manipulation during HB
Browse files Browse the repository at this point in the history
There was a bug where in fused mode we are not also setting the 2nd
half of the fused core to have the appropriate scom values in the fir
action registers. Also we were not re-applying the desired values back
to the core fir action registers at the end of hostboot because of a
simple bug.

Change-Id: I32ae9c7839e27392cfdf0401d008a12941804beb
CQ: SW432725
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/60439
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
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: Elizabeth K. Liner <eliner@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@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
crgeddes authored and dcrowell77 committed Jun 14, 2018
1 parent d50263f commit e15b65e
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 2 deletions.
17 changes: 16 additions & 1 deletion src/usr/isteps/istep16/call_host_activate_master.C
Original file line number Diff line number Diff line change
Expand Up @@ -371,10 +371,25 @@ void* call_host_activate_master (void *io_pArgs)
if( l_errl )
{
TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace,
"core_checkstop_helper_hwp ERROR: returning.");
"core_checkstop_helper_hwp on master ERROR: returning.");
break;
}

// We want to make sure the fused pair is also setting the
// core firs to handle checkstops at system level and not
// at the local level
if(l_fusedCore != NULL)
{
l_errl = HBPM::core_checkstop_helper_hwp(l_fusedCore, true);

if ( l_errl )
{
TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace,
"core_checkstop_helper_hwp on fused pair ERROR: returning.");
break;
}
}

// Take new checkstop values and insert them into the homer image
l_errl = HBPM::core_checkstop_helper_homer();

Expand Down
2 changes: 1 addition & 1 deletion src/usr/isteps/istep21/call_host_start_payload.C
Original file line number Diff line number Diff line change
Expand Up @@ -405,7 +405,7 @@ void* call_host_start_payload (void *io_pArgs)
if(! is_sapphire_load() )
{
TARGETING::TargetHandleList l_coreTargetList;
getAllChips(l_coreTargetList, TYPE_CORE);
getAllChiplets(l_coreTargetList, TYPE_CORE);

for( auto l_core_target : l_coreTargetList)
{
Expand Down

0 comments on commit e15b65e

Please sign in to comment.