diff options
author | Christian Geddes <crgeddes@us.ibm.com> | 2017-10-27 16:38:35 -0500 |
---|---|---|
committer | William G. Hoffa <wghoffa@us.ibm.com> | 2017-10-31 14:33:06 -0400 |
commit | ac7f3d7b4b95342de5a692b623dba7ae4b875ec4 (patch) | |
tree | ea081be6b129458a89e2247b34e2150333151c50 /src/usr/fapi2 | |
parent | 0abd9ab92d13875ab7898847fde9e985279512b8 (diff) | |
download | talos-hostboot-ac7f3d7b4b95342de5a692b623dba7ae4b875ec4.tar.gz talos-hostboot-ac7f3d7b4b95342de5a692b623dba7ae4b875ec4.zip |
Re-enable fapi2 test case module
We determined the problem to be with the getVPD test case when
we tried to get the section info for the MEMD pnor section the
section was not currently loaded. The reason this was sometimes
passing was because the PNOR tests were loading and unloading
the MEMD module. My guess was that we were getting lucking
and reading MEMD while the pnor test has the section loaded
Change-Id: I50c12993d639554ab919ec01bba45e5f4aae761c
RTC: 181401
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/48955
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>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Diffstat (limited to 'src/usr/fapi2')
-rw-r--r-- | src/usr/fapi2/test/fapi2GetVpdTest.H | 42 |
1 files changed, 34 insertions, 8 deletions
diff --git a/src/usr/fapi2/test/fapi2GetVpdTest.H b/src/usr/fapi2/test/fapi2GetVpdTest.H index a72acedbe..8a8dc0849 100644 --- a/src/usr/fapi2/test/fapi2GetVpdTest.H +++ b/src/usr/fapi2/test/fapi2GetVpdTest.H @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2016 */ +/* Contributors Listed Below - COPYRIGHT 2016,2017 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -35,6 +35,9 @@ #include <cxxtest/TestSuite.H> #include <getVpdTest.H> +#include <pnor/pnorif.H> +#include <errl/errlentry.H> +#include <errl/errlmanager.H> class GetVPDTest : public CxxTest::TestSuite { @@ -43,19 +46,42 @@ public: void testGetVPD(void) { - testGetVPD_MR(); +#ifdef CONFIG_SECUREBOOT + errlHndl_t pError=NULL; + do + { + pError = PNOR::loadSecureSection(PNOR::MEMD); + if(pError) + { + TS_FAIL("testGetVPD:: failed to load MEMD secure section"); + ERRORLOG::errlCommit(pError,PNOR_COMP_ID); + break; + } +#endif + testGetVPD_MR(); + + testGetVPD_MT(); - testGetVPD_MT(); + testDecode_MR(); - testDecode_MR(); + testDecode_MT(); - testDecode_MT(); + testGetVPD_Override(); - testGetVPD_Override(); + testGetVPD_DQ(); - testGetVPD_DQ(); + testGetVPD_CK(); +#ifdef CONFIG_SECUREBOOT + pError = PNOR::unloadSecureSection(PNOR::MEMD); + if(pError) + { + TS_FAIL("testGetVPD:: failed to unload MEMD secure section"); + ERRORLOG::errlCommit(pError,PNOR_COMP_ID); + break; + } + } while(0); +#endif - testGetVPD_CK(); } }; // GetVPDTest class |