diff options
author | Andre A. Marin <aamarin@us.ibm.com> | 2019-04-26 16:41:53 -0500 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2019-04-30 11:55:05 -0500 |
commit | 3b13a6483b18a93d67b36c5a28d07415212a6995 (patch) | |
tree | ef114a2ef722e7a3b355b5b0738d49b560839614 | |
parent | 114d10d29c181671b0ec51c1bef37652a2df9b83 (diff) | |
download | talos-hostboot-3b13a6483b18a93d67b36c5a28d07415212a6995.tar.gz talos-hostboot-3b13a6483b18a93d67b36c5a28d07415212a6995.zip |
Add call to setup derived attributes in p9a_mss_eff_config
Previously there was some work done to write the functions that
do the calculations for these derived attributes but the code
was not being called, and one of the functions had an incorrect
parameter. This commit addresses both of those issues and also
improves on a trace in eff_grouping that made debugging this issue
tricky.
Change-Id: I8394f1dcc29374afc765668fce79e55f9f823e7d
RTC: 207850
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/76616
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/76661
Reviewed-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Tested-by: Christian R. Geddes <crgeddes@us.ibm.com>
-rw-r--r-- | src/import/chips/p9/procedures/hwp/nest/p9_mss_eff_grouping.C | 2 | ||||
-rw-r--r-- | src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C | 4 |
2 files changed, 5 insertions, 1 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_mss_eff_grouping.C b/src/import/chips/p9/procedures/hwp/nest/p9_mss_eff_grouping.C index 342e32247..479a78b58 100644 --- a/src/import/chips/p9/procedures/hwp/nest/p9_mss_eff_grouping.C +++ b/src/import/chips/p9/procedures/hwp/nest/p9_mss_eff_grouping.C @@ -745,7 +745,7 @@ fapi2::ReturnCode EffGroupingMccAttrs::getAttrs( iv_dimmSize = (iv_ocmbs.size() * l_min_size); // Display this OMI's attribute info - FAPI_INF("EffGroupingMccAttrs::getAttrs: MCC %d, OCMBs attached %d, " + FAPI_INF("EffGroupingMccAttrs::getAttrs: MCC %d, OCMBs w/ memory attached %d, " "iv_dimmSize %d GB ", iv_unitPos, iv_ocmbs.size(), iv_dimmSize); diff --git a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C index 3bbf0ea9f..8a5529734 100644 --- a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C +++ b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C @@ -112,6 +112,10 @@ fapi2::ReturnCode p9a_mss_eff_config( const fapi2::Target<fapi2::TARGET_TYPE_MEM FAPI_TRY( mss::attr_eff_engine<mss::exp::attr_eff_engine_fields>::set(l_spd_decoder) ); } } + + // Set up derived ATTRS + FAPI_TRY( mss::attr_derived_engine<mss::attr_engine_derived_fields>::set(dimm) ); + }// dimm fapi_try_exit: |