diff options
author | Joe McGill <jmcgill@us.ibm.com> | 2017-12-01 15:51:11 -0600 |
---|---|---|
committer | Sachin Gupta <sgupta2m@in.ibm.com> | 2017-12-04 22:36:36 -0500 |
commit | 01f19f8482837cf8600ffec2ce9e5a68f50d8e97 (patch) | |
tree | 04ef91c10d041d141ff933ea41d1416904d17d25 /src/import/chips | |
parent | 1f0bc6efb21793e7275d11f42323a8eb00c2e835 (diff) | |
download | talos-sbe-01f19f8482837cf8600ffec2ce9e5a68f50d8e97.tar.gz talos-sbe-01f19f8482837cf8600ffec2ce9e5a68f50d8e97.zip |
p9_mss_eff_grouping -- fix mirrored memory mapping bug introduced by 49290
p9_fbc_utils.C
return each mirorred region as two chunks, to make
m.size = nm0.size + nm1.size for Cumulus
p9_mss_eff_grouping.C
fix off by one error in max region index calculation
Change-Id: I79ac3651de048ed83a0eff704b49b25bff9a84cb
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/50393
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Dev-Ready: Joseph J. McGill <jmcgill@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
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: JEREMY R. NEATON <jrneaton@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/50397
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Diffstat (limited to 'src/import/chips')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/nest/p9_fbc_utils.C | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_fbc_utils.C b/src/import/chips/p9/procedures/hwp/nest/p9_fbc_utils.C index aa69a0b8..a1fd87bc 100644 --- a/src/import/chips/p9/procedures/hwp/nest/p9_fbc_utils.C +++ b/src/import/chips/p9/procedures/hwp/nest/p9_fbc_utils.C @@ -352,8 +352,13 @@ fapi2::ReturnCode p9_fbc_utils_get_chip_base_address( { o_base_address_nm0.push_back(l_base_address_nm0 | l_alias_mask()); - o_base_address_m.push_back(l_base_address_m | - l_alias_mask()); + // first half + o_base_address_m.push_back((l_base_address_m | + l_alias_mask()) + 0); + // second half + o_base_address_m.push_back((l_base_address_m | + l_alias_mask()) + + (MAX_INTERLEAVE_GROUP_SIZE / 2)); } // second non-mirrored msel region unusable with HW423589_OPTION2 |