From b0d5f27a21da630c3eb9072692e7b72cff134d2e Mon Sep 17 00:00:00 2001 From: Marty Gloff Date: Tue, 26 Sep 2017 14:04:12 -0500 Subject: [PATCH] Traces from bringup memory remapping for no memory behind master proc During debug of problems with OP / BMC systems, a trace bug was found and some debug traces were used. This commit captures select trace changes. Change-Id: I9398c0b0ccd44414e6fc360cfc68db581729db6a RTC: 149250 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/46769 Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Tested-by: Jenkins OP Build CI Reviewed-by: Matt Derksen Tested-by: Jenkins OP HW Reviewed-by: Corey V. Swenson Reviewed-by: Daniel M. Crowell --- src/usr/isteps/istep07/call_mss_attr_update.C | 39 +++++++++++++++++++ src/usr/targeting/targetservicestart.C | 3 +- 2 files changed, 41 insertions(+), 1 deletion(-) diff --git a/src/usr/isteps/istep07/call_mss_attr_update.C b/src/usr/isteps/istep07/call_mss_attr_update.C index 4f625ae132e..d548fff84af 100644 --- a/src/usr/isteps/istep07/call_mss_attr_update.C +++ b/src/usr/isteps/istep07/call_mss_attr_update.C @@ -159,6 +159,19 @@ errlHndl_t check_proc0_memory_config(IStepError & io_istepErr) l_proc0 = i; } + TRACDCOMP(ISTEPS_TRACE::g_trac_isteps_trace, + "check_proc0_memory_config: Initial settings for " + "Proc %.8X\n" + " groupIdDflt = %d, groupIdEff = %d, groupId = %d\n" + " chipIdDflt = %d, chipIdEff = %d, chipId = %d", + get_huid(l_procIds[i].proc), + l_procIds[i].groupIdDflt, + l_procIds[i].groupIdEff, + l_procIds[i].groupId, + l_procIds[i].chipIdDflt, + l_procIds[i].chipIdEff, + l_procIds[i].chipId); + // Increment index i++; } @@ -300,6 +313,17 @@ errlHndl_t check_proc0_memory_config(IStepError & io_istepErr) // Loop through all procs detecting that IDs are set correctly for (i = 0; i < l_procsList.size(); i++) { + TRACDCOMP(ISTEPS_TRACE::g_trac_isteps_trace, + "check_proc0_memory_config: Compare settings for " + "Proc %.8X\n" + " groupIdEff = %d, groupId = %d\n" + " chipIdEff = %d, chipId = %d", + get_huid(l_procIds[i].proc), + l_procIds[i].groupIdEff, + l_procIds[i].groupId, + l_procIds[i].chipIdEff, + l_procIds[i].chipId); + if((l_procIds[i].groupId != l_procIds[i].groupIdEff) || (l_procIds[i].chipId != l_procIds[i].chipIdEff) ) { @@ -311,6 +335,21 @@ errlHndl_t check_proc0_memory_config(IStepError & io_istepErr) l_updateNeeded = true; } + + TRACDCOMP(ISTEPS_TRACE::g_trac_isteps_trace, + "check_proc0_memory_config: Current attribute " + "settings for Proc %.8X\n" + " ATTR_PROC_EFF_FABRIC_GROUP_ID = %d\n" + " ATTR_FABRIC_GROUP_ID = %d\n" + " ATTR_PROC_EFF_FABRIC_CHIP_ID = %d\n" + " ATTR_FABRIC_CHIP_ID = %d", + get_huid(l_procIds[i].proc), + (l_procIds[i].proc)-> + getAttr(), + (l_procIds[i].proc)->getAttr(), + (l_procIds[i].proc)-> + getAttr(), + (l_procIds[i].proc)->getAttr()); } if(l_updateNeeded) diff --git a/src/usr/targeting/targetservicestart.C b/src/usr/targeting/targetservicestart.C index 4ced5dd2de6..4c06c6f520c 100755 --- a/src/usr/targeting/targetservicestart.C +++ b/src/usr/targeting/targetservicestart.C @@ -452,7 +452,8 @@ static void adjustMemoryMap( TargetService& i_targetService ) // Grab the value of the BARs that SBE booted with uint64_t l_curXscomBAR = g_BlToHbDataManager.getXscomBAR(); uint64_t l_curLpcBAR = g_BlToHbDataManager.getLpcBAR(); - TARG_INF( "adjustMemoryMap> xscom=%X, lpc=%X", l_curXscomBAR, l_curLpcBAR ); + TARG_INF( "adjustMemoryMap> xscom=%llX, lpc=%llX", + l_curXscomBAR, l_curLpcBAR ); // Get the master proc Target* l_pMasterProcChip = nullptr;