diff options
author | Greg Still <stillgs@us.ibm.com> | 2017-06-21 15:02:36 -0500 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2017-07-07 18:48:16 -0400 |
commit | aaee2d26ef1b3edf01b40e82336498a9811b04be (patch) | |
tree | f9cafc3ed720dc8cf7b5dcdd0e8894e2943bd195 /src | |
parent | 7846953d646b34254d6d73c8b394f8ca6b02e703 (diff) | |
download | talos-hostboot-aaee2d26ef1b3edf01b40e82336498a9811b04be.tar.gz talos-hostboot-aaee2d26ef1b3edf01b40e82336498a9811b04be.zip |
PM: fix PSTATE_MODE=OFF istep 15 failure
Change-Id: I670a6dfa53eeb1693715d877e7b1446b5ded58a2
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42249
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: BRIAN D. VICTOR <brian.d.victor1@ibm.com>
Reviewed-by: Brian T. Vanderpool <vanderp@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42250
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: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C b/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C index c7428fcb6..992a0c620 100644 --- a/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C +++ b/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C @@ -242,7 +242,13 @@ p9_pstate_parameter_block( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_ //the data. if (attr.attr_pstate_mode == fapi2::ENUM_ATTR_SYSTEM_PSTATES_MODE_OFF) { - FAPI_INF("Requested for not to boot the PGPE, So PPB won't be initialized"); + FAPI_INF("Pstate mode is to not boot the PGPE. Thus, none of the parameter blocks will be constructed"); + + // Set the io_size to 0 so that memory allocation issues won't be + // detected by the caller. + + io_size = 0; + break; } @@ -569,7 +575,8 @@ p9_pstate_parameter_block( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_ uint8_t l_nest_leakage_for_occ = 75; if (attr.attr_system_wof_disable == fapi2::ENUM_ATTR_SYSTEM_WOF_DISABLE_OFF) - { + { io_size = 0; + uint16_t l_iac_tdp_vdn = get_iac_vdn_value (l_vpd_vdn_mv, l_iddqt, l_nest_leakage_for_occ, l_vpd_idn_100ma); @@ -1237,7 +1244,7 @@ proc_get_mvpd_iddq( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_target, strcpy(l_line_str, string); \ sprintf(l_buffer_str, "%-*s", size, l_line_str);\ strcpy(l_line_str, l_buffer_str); \ - strcpy(l_buffer_str, ""); + strcpy(l_buffer_str, ""); // get IVDDQ measurements with all good cores ON IDDQ_TRACE (" IDDQ all good cores ON:", IDDQ_DESC_SIZE); |