diff options
| author | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2020-02-12 22:32:55 +0530 |
|---|---|---|
| committer | Daniel M Crowell <dcrowell@us.ibm.com> | 2020-02-17 09:25:04 -0600 |
| commit | 76571f381d074c99f44d3ff94d4607cc09eef1be (patch) | |
| tree | abd8be68da01da9c69e2d634b2c3b7d84b9234e2 | |
| parent | 46174931e05562140af4ef53b5074420cf142332 (diff) | |
| download | talos-hostboot-76571f381d074c99f44d3ff94d4607cc09eef1be.tar.gz talos-hostboot-76571f381d074c99f44d3ff94d4607cc09eef1be.zip | |
MPIPL: Copy all SBE data to hypervisor memory
This patch fixes couple of regressions introduced by commit d99b1ed.
- Setting wrong offset size. As per HDAT spec internal data structure
array size should be 0x10.
- Remove double increment of procNum
Fixes: d99b1ed (Add SBE Arch dump area for both OPAL & PHYP)
Change-Id: I8a3737c1375b74fb488136a3aefa3a4420cf1b6c
CC: Dean Sanner <dsanner@us.ibm.com>
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/91580
Reviewed-by: MURULIDHAR NATARAJU <murulidhar@in.ibm.com>
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>
| -rw-r--r-- | src/usr/dump/dumpCollect.C | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/usr/dump/dumpCollect.C b/src/usr/dump/dumpCollect.C index 970231213..b85bcd28b 100644 --- a/src/usr/dump/dumpCollect.C +++ b/src/usr/dump/dumpCollect.C @@ -421,7 +421,6 @@ errlHndl_t copyArchitectedRegs(void) uint64_t procSrcAddr = (reinterpret_cast<uint64_t>(vMapSrcAddrBase)+ procNum * VMM_ARCH_REG_DATA_PER_PROC_SIZE); TRACDCOMP(g_trac_dump, "SBE Proc[%d] [%p]", procNum, procSrcAddr); - procNum++; sbeArchRegDumpProcHdr_t *sbeProcHdr = reinterpret_cast<sbeArchRegDumpProcHdr_t *>(procSrcAddr); @@ -503,7 +502,7 @@ errlHndl_t copyArchitectedRegs(void) hostHdr->pir = sbeTdHdr->pir; hostHdr->coreState = sbeTdHdr->coreState; hostHdr->iv_regArrayHdr.hdatOffset = - sizeof(hostArchRegDataHdr); + sizeof(HDAT::hdatHDIFDataArray_t); hostHdr->iv_regArrayHdr.hdatArrayCnt = regCount; hostHdr->iv_regArrayHdr.hdatAllocSize = sizeof(hostArchRegDataEntry); |
