diff options
author | Chris Cain <cjcain@us.ibm.com> | 2017-11-20 13:51:39 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-11-22 10:25:12 -0500 |
commit | 4e72331566db512e82f4d320d627ed9866967a5e (patch) | |
tree | 2731eb03b1bddf6021affd072f170043c5f66bae /src/usr/htmgt | |
parent | a420f6c23b46b1e582825035999e6f48f1059cb1 (diff) | |
download | talos-hostboot-4e72331566db512e82f4d320d627ed9866967a5e.tar.gz talos-hostboot-4e72331566db512e82f4d320d627ed9866967a5e.zip |
HTMGT: Do not validate throttles based on oversubscription
Change-Id: I20520c38ce791a63a1f8a43e9e1afdeb8596b63f
CQ: SW408496
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/49921
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Sheldon R. Bailey <baileysh@us.ibm.com>
Reviewed-by: Christopher J. Cain <cjcain@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/htmgt')
-rw-r--r-- | src/usr/htmgt/htmgt_memthrottles.C | 16 | ||||
-rw-r--r-- | src/usr/htmgt/test/htmgtcfgtest.H | 8 |
2 files changed, 4 insertions, 20 deletions
diff --git a/src/usr/htmgt/htmgt_memthrottles.C b/src/usr/htmgt/htmgt_memthrottles.C index e4c5593f1..0a1fafa13 100644 --- a/src/usr/htmgt/htmgt_memthrottles.C +++ b/src/usr/htmgt/htmgt_memthrottles.C @@ -361,25 +361,9 @@ errlHndl_t memPowerThrottlePowercap( mcs_target->tryGetAttr<ATTR_N_PLUS_ONE_N_PER_MBA>(l_slot_redun); mcs_target->tryGetAttr<ATTR_N_PLUS_ONE_N_PER_CHIP>(l_port_redun); mcs_target->tryGetAttr<ATTR_N_PLUS_ONE_MEM_POWER>(l_power_redun); - ATTR_OVERSUB_N_PER_MBA_type l_slot_oversub = {0}; - ATTR_OVERSUB_N_PER_CHIP_type l_port_oversub = {0}; - ATTR_OVERSUB_MEM_POWER_type l_power_oversub = {0}; - mcs_target->tryGetAttr<ATTR_OVERSUB_N_PER_MBA>(l_slot_oversub); - mcs_target->tryGetAttr<ATTR_OVERSUB_N_PER_CHIP>(l_port_oversub); - mcs_target->tryGetAttr<ATTR_OVERSUB_MEM_POWER>(l_power_oversub); unsigned int mca_index; for (mca_index = 0; mca_index < TMGT_MAX_MCA_PER_MCS; ++mca_index) { - if (l_slot[mca_index] > l_slot_oversub[mca_index]) - { - TMGT_INF("memPowerThrottlePowercap: MCS%d/MCA%d" - " oversub throttle (%d) < pcap throttle (%d)", - mcs_unit, mca_index, - l_slot_oversub[mca_index], l_slot[mca_index]); - l_slot[mca_index] = l_slot_oversub[mca_index]; - l_port[mca_index] = l_port_oversub[mca_index]; - l_power[mca_index] = l_power_oversub[mca_index]; - } if (l_slot[mca_index] > l_slot_redun[mca_index]) { TMGT_INF("memPowerThrottlePowercap: MCS%d/MCA%d - " diff --git a/src/usr/htmgt/test/htmgtcfgtest.H b/src/usr/htmgt/test/htmgtcfgtest.H index ab71ec034..3ac3f82fc 100644 --- a/src/usr/htmgt/test/htmgtcfgtest.H +++ b/src/usr/htmgt/test/htmgtcfgtest.H @@ -149,17 +149,17 @@ public: (*mba)->getAttr<ATTR_HUID>()); } - (*mba)->tryGetAttr<ATTR_OVERSUB_N_PER_MBA>(n); + (*mba)->tryGetAttr<ATTR_POWERCAP_N_PER_MBA>(n); if ((0 == n[0]) && (0 == n[1])) { - TS_FAIL("MBA 0x%X has value of zero for OVERSUB_N_PER_MBA", + TS_FAIL("MBA 0x%X has value of zero for POWERCAP_N_PER_MBA", (*mba)->getAttr<ATTR_HUID>()); } - (*mba)->tryGetAttr<ATTR_OVERSUB_N_PER_CHIP>(n); + (*mba)->tryGetAttr<ATTR_POWERCAP_N_PER_CHIP>(n); if ((0 == n[0]) && (0 == n[1])) { - TS_FAIL("MBA 0x%X has value of zero for OVERSUB_N_PER_CHIP", + TS_FAIL("MBA 0x%X has value of zero for POWERCAP_N_PER_CHIP", (*mba)->getAttr<ATTR_HUID>()); } } |