diff options
| author | crgeddes <crgeddes@us.ibm.com> | 2017-04-25 10:47:27 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-04-26 16:05:57 -0400 |
| commit | ec1b9e5daacce95f00521fdc3af538e632709ade (patch) | |
| tree | 6f4622e27d1cce8da381a3ddb399ad0564415b65 /src/usr/fapi2 | |
| parent | e9125b17114535f6b18710f081747a72c8da3f23 (diff) | |
| download | talos-hostboot-ec1b9e5daacce95f00521fdc3af538e632709ade.tar.gz talos-hostboot-ec1b9e5daacce95f00521fdc3af538e632709ade.zip | |
Ensure correct # of EX returned from fapi2::proc.getChildren()
There was a story saying we were getting the wrong # of EX targets
back when we called fapi2's getChildren on a nimbus proc trying
to get EX targets. I don't think that we actually were getting wrong
Change-Id: I0b7643cde4bd791aaba4e6956c5ade1f1275f294
RTC:161587
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39655
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/fapi2')
| -rw-r--r-- | src/usr/fapi2/test/fapi2GetChildrenTest.H | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/src/usr/fapi2/test/fapi2GetChildrenTest.H b/src/usr/fapi2/test/fapi2GetChildrenTest.H index 3a35113c1..15ca26fe4 100644 --- a/src/usr/fapi2/test/fapi2GetChildrenTest.H +++ b/src/usr/fapi2/test/fapi2GetChildrenTest.H @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2015,2016 */ +/* Contributors Listed Below - COPYRIGHT 2015,2017 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -137,7 +137,7 @@ void test_fapi2GetChildren() std::vector<Target<fapi2::TARGET_TYPE_CORE> > l_childCores; std::vector<Target<fapi2::TARGET_TYPE_MCA> > l_childMCAs; std::vector<Target<fapi2::TARGET_TYPE_EQ> > l_childEQs; - std::vector<Target<fapi2::TARGET_TYPE_XBUS> > l_childXBUSs; + std::vector<Target<fapi2::TARGET_TYPE_EX> > l_childEXs; // @TODO RTC:148761 // Need to figure out how we are going to set up this relationship // l_childMCAs = fapi2_mcbistTarget.getChildren<fapi2::TARGET_TYPE_MCA>(TARGET_STATE_PRESENT); @@ -225,6 +225,21 @@ void test_fapi2GetChildren() break; } + //Explicitly make sure getChildren for EX targets works + l_childEXs = fapi2_procTarget.getChildren<fapi2::TARGET_TYPE_EX>(TARGET_STATE_PRESENT); + l_targetHuid = TARGETING::get_huid(l_nimbusProc) ; + l_actualSize = l_childEXs.size(); + + //Set expected size to be the number of Exs per proc + l_expectedSize = EQ_PER_PROC * EX_PER_EQ; + numTests++; + if(l_actualSize != l_expectedSize) + { + FAPI_INF("fapi2TargetTest:: EXs per proc mismatch"); + numFails++; + break; + } + // Test pervasive children // Valid children for PERV: |
