diff options
author | Matt Derksen <mderkse1@us.ibm.com> | 2018-08-14 17:21:18 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-08-18 13:11:44 -0500 |
commit | 14340c147361cc0616582b090f7abcf1901072f6 (patch) | |
tree | 73fc20ecd635da755d6fbf7dece44a28e7792471 /src/usr/isteps/tod | |
parent | b2ff2dd0c8e5ced78c84cb5f58adf3c3248369fc (diff) | |
download | talos-hostboot-14340c147361cc0616582b090f7abcf1901072f6.tar.gz talos-hostboot-14340c147361cc0616582b090f7abcf1901072f6.zip |
Use ATTR_MAX_COMPUTE_NODES_PER_SYSTEM instead of hardcoding
hwsvd crash observed during handling hbrt message to get
tod topology data because allocated size was based on a
hardcoded value for max_compute_nodes_per_system
Change-Id: I587dac32fab9dd67fe96e213c5ef72626e4e2ea1
CQ:SW434860
CMVC-Coreq: 1059506
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64459
Reviewed-by: Roland Veloz <rveloz@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/isteps/tod')
-rw-r--r-- | src/usr/isteps/tod/TodUtils.C | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/src/usr/isteps/tod/TodUtils.C b/src/usr/isteps/tod/TodUtils.C index a1a1ede63..493787861 100644 --- a/src/usr/isteps/tod/TodUtils.C +++ b/src/usr/isteps/tod/TodUtils.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2016,2017 */ +/* Contributors Listed Below - COPYRIGHT 2016,2018 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -161,14 +161,8 @@ errlHndl_t getMaxConfigParams( o_maxConfigParams.max_mcs_per_sys = l_pTopLevel->getAttr < TARGETING::ATTR_MAX_MCS_PER_SYSTEM > (); - // TODO RTC 181481: attribute ATTR_MAX_COMPUTE_NODES_PER_SYSTEM is - // currently not implemented as needed to do this assignment. I was - // assured (10/2017) that HB only supports one node, so I am defaulting - // this to 1. In the future HB will need to be multi-node aware and - // TARGETING::ATTR_MAX_COMPUTE_NODES_PER_SYSTEM implemented. - o_maxConfigParams.max_compute_nodes_per_sys = 1; -// o_maxConfigParams.max_compute_nodes_per_sys = l_pTopLevel->getAttr -// < TARGETING::ATTR_MAX_COMPUTE_NODES_PER_SYSTEM > (); + o_maxConfigParams.max_compute_nodes_per_sys = l_pTopLevel->getAttr + < TARGETING::ATTR_MAX_COMPUTE_NODES_PER_SYSTEM > (); } while(0); |