diff options
author | Thi Tran <thi@us.ibm.com> | 2016-08-19 10:53:30 -0500 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2019-02-28 09:40:11 -0600 |
commit | ebd72d043f2358485d2a1ad293c2ffbfc3369dba (patch) | |
tree | 797a6dd66276169f4a5ab28579635ed14c0a0fff /src/import/chips/p9/procedures/hwp | |
parent | 888662cc151b2b3e2eb24540e82f527230df70cb (diff) | |
download | talos-hostboot-ebd72d043f2358485d2a1ad293c2ffbfc3369dba.tar.gz talos-hostboot-ebd72d043f2358485d2a1ad293c2ffbfc3369dba.zip |
Change ATTR_MRW_ENHANCED_GROUPING_NO_MIRRORING name
Change to ATTR_MRW_HW_MIRRORING_ENABLE and invert its meaning to
make it more straight forward.
Change-Id: I9c4fbd30d7832d2b4a11e0a5420d52f79cfb41a6
Original-Change-Id: I0e63d52285a10bb1e8c75d4ff2598f0c50eab6de
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28536
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: CHRISTINA L. GRAVES <clgraves@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Matt K. Light <mklight@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/72544
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-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>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C b/src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C index 1b3f00701..e384055cd 100644 --- a/src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C +++ b/src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C @@ -74,7 +74,7 @@ fapi2::ReturnCode p9_query_mssinfo(const std::vector<fapi2::Target<fapi2::TARGET uint64_t occ_base; uint64_t occ_size; uint8_t mirror_policy; - uint8_t no_mirror; + uint8_t l_mirrorEnabled; char chipid[200]; char l_target_string[fapi2::MAX_ECMD_STRING_LEN]; @@ -196,14 +196,14 @@ fapi2::ReturnCode p9_query_mssinfo(const std::vector<fapi2::Target<fapi2::TARGET (uint64_t)fapi2::current_err); //Mirrored memory attributes read if enabled - FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_MRW_ENHANCED_GROUPING_NO_MIRRORING, - i_vect_pu_targets[i].getParent<fapi2::TARGET_TYPE_SYSTEM>(), no_mirror), - "Error reading ATTR_MRW_ENHANCED_GROUPING_NO_MIRRORING, l_rc 0x%.8X", + FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_MRW_HW_MIRRORING_ENABLE, + i_vect_pu_targets[i].getParent<fapi2::TARGET_TYPE_SYSTEM>(), l_mirrorEnabled), + "Error reading ATTR_MRW_HW_MIRRORING_ENABLE, l_rc 0x%.8X", (uint64_t)fapi2::current_err); - FAPI_DBG("p9_query_mssinfo: Current no_mirror=%u!\n", no_mirror); + FAPI_DBG("p9_query_mssinfo: Current l_mirrorEnabled=%u!\n", l_mirrorEnabled); - if (no_mirror == 0) + if (l_mirrorEnabled == 1) { // ATTR_PROC_MIRROR_SIZES FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_PROC_MIRROR_SIZES, i_vect_pu_targets[i], mirror_sizes), @@ -293,7 +293,7 @@ fapi2::ReturnCode p9_query_mssinfo(const std::vector<fapi2::Target<fapi2::TARGET // Print out memory reservations //------------------------------------------------------------------------------------------------------------------------ if(((mirror_policy == fapi2::ENUM_ATTR_MEM_MIRROR_PLACEMENT_POLICY_NORMAL ) || - (mirror_policy == fapi2::ENUM_ATTR_MEM_MIRROR_PLACEMENT_POLICY_FLIPPED) ) && (no_mirror == 0) ) + (mirror_policy == fapi2::ENUM_ATTR_MEM_MIRROR_PLACEMENT_POLICY_FLIPPED) ) && (l_mirrorEnabled == 1) ) { // ATTR_PROC_NHTM_BAR_BASE_ADDR: Get Nest Hardware Trace Macro (NHTM) bar base addr FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_PROC_NHTM_BAR_BASE_ADDR, i_vect_pu_targets[i], nhtm_base), |