Skip to content

Commit

Permalink
Fixing eff_config_thermal wrapper, added error xml
Browse files Browse the repository at this point in the history
Change-Id: I88fb4a8ff113b2b7539190dc481fcdbe00a8cbd5
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35354
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: Brian R. Silver <bsilver@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35397
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
  • Loading branch information
JacobHarvey authored and crgeddes committed Mar 15, 2017
1 parent 50309ba commit 50b826d
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 1 deletion.
Expand Up @@ -742,6 +742,16 @@ fapi2::ReturnCode set_runtime_m_and_watt_limit( const std::vector< fapi2::Target
//Calculate max power available / number of dimms configured on the VDDR rail
l_watt_target = (l_vmem_power_limit_dimm * l_max_dimms) / l_count_dimms_vec;

// If we have too many dimms, deconfigure the first MCS
// We know there are MCSs on the vector due to the check above
FAPI_ASSERT( (l_count_dimms_vec <= l_max_dimms),
fapi2::MSS_DIMM_COUNT_EXCEEDS_VMEM_REGULATOR_LIMIT()
.set_MAX_DIMM_AMOUNT(l_max_dimms)
.set_DIMMS_SEEN(l_count_dimms_vec),
"The number of dimms counted (%d) on the vector of MCS surpasses the limit (%d)",
l_count_dimms_vec,
l_max_dimms);

FAPI_INF("Calculated ATTR_MSS_MEM_WATT_TARGET is %d, power_limit dimm is %d, max_dimms is %d, count dimms on vector is %d",
l_watt_target,
l_vmem_power_limit_dimm,
Expand Down
Expand Up @@ -217,5 +217,17 @@
</callout>
</hwpError>


<hwpError>
<rc>RC_MSS_DIMM_COUNT_EXCEEDS_VMEM_REGULATOR_LIMIT</rc>
<description>
The number of dimms counted on the vector of MCS passed into p9_mss_eff_config_thermal
exceeds the limit defined in ATTR_MSS_MRW_MAX_NUMBER_DIMMS_POSSIBLE_PER_VMEM_REGULATOR
</description>
<ffdc>MAX_DIMM_AMOUNT</ffdc>
<ffdc>DIMMS_SEEN</ffdc>
<callout>
<procedure>CODE</procedure>
<priority>HIGH</priority>
</callout>
</hwpError>
</hwpErrors>

0 comments on commit 50b826d

Please sign in to comment.