diff options
| author | Dan Crowell <dcrowell@us.ibm.com> | 2017-05-31 10:43:41 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-06-07 12:40:42 -0400 |
| commit | 47f343509fdd57dfe6d2f7e230413db9ebab0066 (patch) | |
| tree | 305a4724b8f20cc6a067784bcc15958e22fe467c /src/include | |
| parent | 94eff06a67f84e9eb0f2189efd7a0b47029fd154 (diff) | |
| download | talos-hostboot-47f343509fdd57dfe6d2f7e230413db9ebab0066.tar.gz talos-hostboot-47f343509fdd57dfe6d2f7e230413db9ebab0066.zip | |
Return only present DIMM children to fapi
Switching from possible to present for fapi2::getChildren
when asked about DIMMs
Change-Id: If5febce96b546ad1553dad0f5feb37ca07c4da97
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41274
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: JACOB L. HARVEY <jlharvey@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/usr/fapi2/target.H | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/src/include/usr/fapi2/target.H b/src/include/usr/fapi2/target.H index 314801ca8..92bc43e18 100644 --- a/src/include/usr/fapi2/target.H +++ b/src/include/usr/fapi2/target.H @@ -784,6 +784,7 @@ FAPI_DBG(ENTER_MRK "getChildren. Type 0x%08x State:0x%08x", T, i_state); if(!l_functional) { + // PERV targets use a special interface if(K == fapi2::TARGET_TYPE_PERV) { TARGETING::getPervasiveChildTargetsByState(l_childList, @@ -792,6 +793,18 @@ FAPI_DBG(ENTER_MRK "getChildren. Type 0x%08x State:0x%08x", T, i_state); l_type, TARGETING::UTIL_FILTER_ALL); } + // DIMMs need to use PRESENT so that we don't report things + // that aren't installed + else if(K == fapi2::TARGET_TYPE_DIMM) + { + TARGETING::getChildAffinityTargetsByState(l_childList, + static_cast<TARGETING::Target*>(this->get()), + TARGETING::CLASS_NA, + l_type, + TARGETING::UTIL_FILTER_PRESENT); + } + // All chiplets need to use ALL so that we report the architectural + // limits, versus what PG might say else { TARGETING::getChildAffinityTargetsByState(l_childList, |
