Skip to content

Commit

Permalink
MPIPL: ZZ-L, Update copyArchitectedRegs() for non-HW dump flow
Browse files Browse the repository at this point in the history
FSP has a non-HW dump collection MPIPL flow. In that flow SBE is
not called to collect the spr/gpr registers, Modified
copyArchitectedRegs() to skip validation and copying of data in
that flow.

CQ:SW485576
Change-Id: If8849d25163a92b72b8fbc6b205eaf98b1a6b80f
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/92297
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>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
  • Loading branch information
nmuruli authored and dcrowell77 committed Feb 26, 2020
1 parent 17867d0 commit d348d7c
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions src/usr/dump/dumpCollect.C
Expand Up @@ -427,6 +427,18 @@ errlHndl_t copyArchitectedRegs(void)
uint16_t threadCount = sbeProcHdr->thread_cnt;
uint16_t regCount = sbeProcHdr->reg_cnt;

//FSP has a non-HW dump collection MPIPL flow. In that flow SBE is
//not called to collect the spr/gpr.There is no way for hostboot to
//know , its non-hw dump mode of MPIPL. Hence check the thread count
//and skip copying of the data.
if(threadCount == 0)
{
TRACFCOMP(g_trac_dump, "copyArchitectedRegs(): Data not not "
"collected by SBE coreCount=%d threadCount=%d and regCount=%d",
sbeProcHdr->core_cnt,threadCount,regCount);
continue;
}

//Validate the structure versions used by SBE and HB for sharing the
//data
if( sbeProcHdr->version != REG_DUMP_SBE_HB_STRUCT_VER )
Expand Down

0 comments on commit d348d7c

Please sign in to comment.