diff options
author | Thi Tran <thi@us.ibm.com> | 2014-05-05 15:32:53 -0500 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2014-05-08 16:07:00 -0500 |
commit | 61e77208b99c4086e9da6b313b9b9e4d9d79348f (patch) | |
tree | eaa0a8fdb18aec8e285bb5e57a99189e3a5da556 /src/usr/hwpf/hwp/pstates | |
parent | 679df937c8d61247dfff4c0cbe21d7a8af43e7b0 (diff) | |
download | talos-hostboot-61e77208b99c4086e9da6b313b9b9e4d9d79348f.tar.gz talos-hostboot-61e77208b99c4086e9da6b313b9b9e4d9d79348f.zip |
SW258328: FW820-FTC P8: Apparent hot/cold counter data corrupting memory outside
Change-Id: Ie5561d14998e735dc53d6a79fd72200fc4ae38ae
CQ:SW258328
Depends-On:I9276f5e2cabf4fa0cbad294d8d683cf53799f9b4
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/10972
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Tested-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/10973
Tested-by: Jenkins Server
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/usr/hwpf/hwp/pstates')
-rwxr-xr-x | src/usr/hwpf/hwp/pstates/pstates/p8_build_pstate_datablock.C | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/usr/hwpf/hwp/pstates/pstates/p8_build_pstate_datablock.C b/src/usr/hwpf/hwp/pstates/pstates/p8_build_pstate_datablock.C index 405e253fb..6b785d14c 100755 --- a/src/usr/hwpf/hwp/pstates/pstates/p8_build_pstate_datablock.C +++ b/src/usr/hwpf/hwp/pstates/pstates/p8_build_pstate_datablock.C @@ -20,7 +20,7 @@ /* Origin: 30 */ /* */ /* IBM_PROLOG_END_TAG */ -// $Id: p8_build_pstate_datablock.C,v 1.34 2014/04/07 02:17:52 stillgs Exp $ +// $Id: p8_build_pstate_datablock.C,v 1.35 2014/04/09 22:00:46 stillgs Exp $ // $Source: /afs/awd/projects/eclipz/KnowledgeBase/.cvsroot/eclipz/chips/p8/working/procedures/ipl/fapi/p8_build_pstate_datablock.C,v $ //------------------------------------------------------------------------------ // *! (C) Copyright International Business Machines Corp. 2012 @@ -1078,6 +1078,7 @@ ReturnCode proc_boost_gpst (PstateSuperStructure *pss, uint8_t gpsi_max; const uint32_t MAXIMUM_BOOST_PERCENT_SUPPORTED = 20; + const uint32_t MAXIMUM_PSTATE_RANGE = 255; // maximum represented by uint8_t do { @@ -1124,7 +1125,7 @@ ReturnCode proc_boost_gpst (PstateSuperStructure *pss, boosted_pct, pstate0_frequency_khz, boosted_freq_khz); break; } - else if (pstate_diff > 255) { + else if (pstate_diff > MAXIMUM_PSTATE_RANGE) { FAPI_ERR("Percentage boost calculation overrun produced invalid Pstate Difference: %u", pstate_diff); const uint32_t& PSTATEDIFF = pstate_diff; const uint32_t& BOOSTEDFREQKHZ = boosted_freq_khz; |