diff options
author | Prasad Bg Ranganath <prasadbgr@in.ibm.com> | 2017-10-03 04:34:33 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-10-09 12:03:17 -0400 |
commit | 37d98e479d900300acccf9dec1453dce7a2cdc5c (patch) | |
tree | a5abf92bb281b269b4dc2ba5c36ff3631325a7c0 /src | |
parent | 3fab32bfe20fa1d94262eee7db80a2fabcd3b1da (diff) | |
download | talos-hostboot-37d98e479d900300acccf9dec1453dce7a2cdc5c.tar.gz talos-hostboot-37d98e479d900300acccf9dec1453dce7a2cdc5c.zip |
PSTATE_PARAMETER_BLOCK: FIx VFRT trace logic
Change-Id: I1caa4fa1dc3d676ca4a718e97539f2c686d216ba
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/47058
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
Reviewed-by: Prem Shanker Jha <premjha2@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/47062
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: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C | 15 |
1 files changed, 12 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 3dc27116a..a9a73ead8 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 @@ -3193,7 +3193,7 @@ iddq_print(IddqTable* i_iddqt) FAPI_INF("%s", l_line_str); - // get average temperatur}e measurements with all cores and caches OFF + // get average temperature measurements with all cores and caches OFF IDDQ_TRACE (" Average temp all cores OFF, caches OFF:", IDDQ_DESC_SIZE); for (i = 0; i < IDDQ_MEASUREMENTS; i++) @@ -3818,13 +3818,14 @@ void p9_pstate_update_vfrt(const GlobalPstateParmBlock* i_gppb, bool b_fratio_set = true; + bool b_first_vratio_set = true; + //Initialize VFRT data part for (l_index_0 = 0; l_index_0 < VFRT_FRATIO_SIZE; ++l_index_0) { strcpy(l_buffer_str, ""); strcpy(l_line_str, " "); - bool b_first_vratio_set = true; for (l_index_1 = 0; l_index_1 < VFRT_VRATIO_SIZE; ++l_index_1) { @@ -3852,7 +3853,7 @@ void p9_pstate_update_vfrt(const GlobalPstateParmBlock* i_gppb, // is correct without overfilling the HB trace buffer. if (!((l_index_1 + 1) % 8) && b_first_vratio_set && b_fratio_set) { - FAPI_INF("%s", l_line_str); + FAPI_DBG("%s ", l_line_str); strcpy(l_buffer_str, ""); strcpy(l_line_str, " "); b_first_vratio_set = false; @@ -3863,7 +3864,15 @@ void p9_pstate_update_vfrt(const GlobalPstateParmBlock* i_gppb, // If fratio is not enabled, don't trace the remaining, duplicate entries. if (!l_enable_fratio) + { b_fratio_set = false; + b_first_vratio_set = false; + } + else + { + b_fratio_set = true; + b_first_vratio_set = true; + } } |