diff options
author | Prem Shanker Jha <premjha2@in.ibm.com> | 2018-12-26 00:20:06 -0600 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2019-01-29 11:27:38 -0600 |
commit | 9c147a336dfa8950ce1ccf0246a8e9d693064b5b (patch) | |
tree | 9755d38dc1064471c25c61c581320d0549b679f5 | |
parent | 39a6b40220d9251f373a1ffa3ebfa379e6fa0b2c (diff) | |
download | talos-hostboot-9c147a336dfa8950ce1ccf0246a8e9d693064b5b.tar.gz talos-hostboot-9c147a336dfa8950ce1ccf0246a8e9d693064b5b.zip |
PM Malf Func: Fixed bugs pertaining pm malfunction FFDC and parser.
Fixed following:
- printing of 64 bit register value.
- collection 64 bit FFDC register.
Key_Cronus_Test=PM_REGRESS
CQ: SW454621
Change-Id: I4f09f635b0232bffc4cefd368f1f004569c5ef83
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70017
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: Cronus HW CI <cronushw-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70018
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: Christian R. Geddes <crgeddes@us.ibm.com>
-rw-r--r-- | src/import/hwpf/fapi2/include/collect_reg_ffdc.H | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/import/hwpf/fapi2/include/collect_reg_ffdc.H b/src/import/hwpf/fapi2/include/collect_reg_ffdc.H index 98f1f3b80..34d9a8edc 100644 --- a/src/import/hwpf/fapi2/include/collect_reg_ffdc.H +++ b/src/import/hwpf/fapi2/include/collect_reg_ffdc.H @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2015,2018 */ +/* Contributors Listed Below - COPYRIGHT 2015,2019 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -417,9 +417,9 @@ ReturnCode collectRegisterAndAddressData(std::vector<T>& i_addresses, U& i_reade { fapi2::buffer<T> l_buf; - T l_address = address + i_offset; + l_addr = address + i_offset; - l_rc = i_reader.read_register(l_address, l_buf); + l_rc = i_reader.read_register(l_addr, l_buf); if(l_rc) { @@ -434,9 +434,7 @@ ReturnCode collectRegisterAndAddressData(std::vector<T>& i_addresses, U& i_reade l_data = l_buf(); } - l_address = htobe64(l_address); - l_data = htobe64(l_data); - l_addr = l_address >> 32; + l_data = htobe64(l_data); memcpy(o_pData, &l_addr, sizeof(uint32_t)); o_pData += sizeof(uint32_t); |