diff options
| author | Louis Stermole <stermole@us.ibm.com> | 2018-01-19 14:14:57 -0600 |
|---|---|---|
| committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2018-01-24 17:38:30 -0500 |
| commit | e75b51ba9604d184bf01b5ea7d572ba614d3d2bb (patch) | |
| tree | 7d04d63ace65978a9412ec27ea11ff55c23b787f /src/import/chips/p9/procedures/hwp/memory/p9_mss_utils_to_throttle.C | |
| parent | cc2d45afa61c873f36d04aa5e4b095321ea677d1 (diff) | |
| download | talos-hostboot-e75b51ba9604d184bf01b5ea7d572ba614d3d2bb.tar.gz talos-hostboot-e75b51ba9604d184bf01b5ea7d572ba614d3d2bb.zip | |
Return failing MCA targets when MSS_CALC_PORT_POWER_EXCEEDS_MAX condition hit
Change-Id: I6e2644eda24d6d47bb06cd52640695f64e47c772
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52284
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Reviewed-by: Michael D. Pardeik <pardeik@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52295
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>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/memory/p9_mss_utils_to_throttle.C')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/p9_mss_utils_to_throttle.C | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/p9_mss_utils_to_throttle.C b/src/import/chips/p9/procedures/hwp/memory/p9_mss_utils_to_throttle.C index 76117735b..eccd56532 100644 --- a/src/import/chips/p9/procedures/hwp/memory/p9_mss_utils_to_throttle.C +++ b/src/import/chips/p9/procedures/hwp/memory/p9_mss_utils_to_throttle.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2015,2017 */ +/* Contributors Listed Below - COPYRIGHT 2015,2018 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -30,8 +30,8 @@ /// allowed within a window of M DRAM clocks given the minimum dram data bus utilization. /// -// *HWP HWP Owner: Jacob Harvey <jlharvey@us.ibm.com> -// *HWP HWP Backup: Andre A. Marin <aamarin@us.ibm.com> +// *HWP HWP Owner: Andre A. Marin <aamarin@us.ibm.com> +// *HWP HWP Backup: Louis Stermole <stermole@us.ibm.com> // *HWP Team: Memory // *HWP Level: 3 // *HWP Consumed by: FSP:HB @@ -71,6 +71,8 @@ extern "C" { FAPI_INF("Entering p9_mss_utils_to_throttle"); + std::vector< fapi2::Target<fapi2::TARGET_TYPE_MCA> > l_exceeded_power; + for( const auto& l_mcs : i_targets ) { if (mss::count_dimm(l_mcs) == 0) @@ -146,8 +148,9 @@ extern "C" l_n_port) ); } - //equalize throttles to prevent variable performance - FAPI_TRY( mss::power_thermal::equalize_throttles (i_targets, mss::throttle_type::POWER)); + // Equalize throttles to prevent variable performance + // Note that we don't do anything with any MCA that exceed the power limit here, as we don't have an input power limit to go from + FAPI_TRY( mss::power_thermal::equalize_throttles (i_targets, mss::throttle_type::POWER, l_exceeded_power)); fapi_try_exit: return fapi2::current_err; |
