summaryrefslogtreecommitdiffstats
path: root/src/usr/hwas
diff options
context:
space:
mode:
authorMatt Raybuck <mraybuc@us.ibm.com>2019-01-16 15:27:34 -0600
committerDaniel M. Crowell <dcrowell@us.ibm.com>2019-01-25 09:46:46 -0600
commitc2f2f5037920dc8441c6b27ff7a488a90f0433b1 (patch)
tree6c573a2204b150b6f1a37e53d7626149a8807405 /src/usr/hwas
parent89a8815250a9a87daaa6db624559fd5b0e465f00 (diff)
downloadtalos-hostboot-c2f2f5037920dc8441c6b27ff7a488a90f0433b1.tar.gz
talos-hostboot-c2f2f5037920dc8441c6b27ff7a488a90f0433b1.zip
remove unnecessary checks in checkMinimumHardware
There were a series of checks that were performed after we had determined the DIMMs were functional that would be implicitly true since the DIMMs are functional. Those were removed. Change-Id: I14feae0129ac96c2f701000b0f265bff36250371 RTC:196804 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70560 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: DHRUVARAJ SUBHASH CHANDRAN <dhruvaraj@in.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/hwas')
-rw-r--r--src/usr/hwas/common/hwas.C267
1 files changed, 0 insertions, 267 deletions
diff --git a/src/usr/hwas/common/hwas.C b/src/usr/hwas/common/hwas.C
index d32378fc3..a112ab2fe 100644
--- a/src/usr/hwas/common/hwas.C
+++ b/src/usr/hwas/common/hwas.C
@@ -2020,273 +2020,6 @@ errlHndl_t checkMinimumHardware(const TARGETING::ConstTargetHandle_t i_nodeOrSys
// errl is now NULL
} // if no dimms
- // There needs to be either functional MCS/MCAs (NIMBUS) or MCS/MBAs
- // (CUMULUS). Check for MCAs first.
- PredicateCTM l_mca(CLASS_UNIT, TYPE_MCA);
-
- TargetHandleList l_presMcaTargetList;
- PredicatePostfixExpr l_checkExprPresMca;
- l_checkExprPresMca.push(&l_mca).push(&l_present).And();
- targetService().getAssociated( l_presMcaTargetList, pTop,
- TargetService::CHILD, TargetService::ALL,
- &l_checkExprPresMca);
-
- PredicateCTM l_membuf(CLASS_CHIP, TYPE_MEMBUF);
-
- TargetHandleList l_presMembufTargetList;
- PredicatePostfixExpr l_checkExprPresMembufs;
- l_checkExprPresMembufs.push(&l_membuf).push(&l_present).And();
- targetService().getAssociated( l_presMembufTargetList, pTop,
- TargetService::CHILD, TargetService::ALL,
- &l_checkExprPresMembufs);
-
-
- PredicateCTM l_ocmb(CLASS_CHIP, TYPE_OCMB_CHIP);
-
- TargetHandleList l_presOcmbTargetList;
- PredicatePostfixExpr l_checkExprPresOcmbs;
- l_checkExprPresOcmbs.push(&l_ocmb).push(&l_present).And();
- targetService().getAssociated( l_presOcmbTargetList, pTop,
- TargetService::CHILD, TargetService::ALL,
- &l_checkExprPresOcmbs);
-
- // If any MCAs are present, then some must be functional
- if (!l_presMcaTargetList.empty())
- {
- TargetHandleList l_funcMcaTargetList;
- PredicatePostfixExpr l_checkExprPresMca;
- l_checkExprPresMca.push(&l_mca).push(&l_functional).And();
- targetService().getAssociated( l_funcMcaTargetList, pTop,
- TargetService::CHILD, TargetService::ALL,
- &l_checkExprPresMca);
-
- HWAS_DBG( "checkMinimumHardware: %d functional MCAs",
- l_funcMcaTargetList.size());
-
- if (l_funcMcaTargetList.empty())
- {
- HWAS_ERR( "Insufficient hardware to continue IPL"
- " (func membufs)");
- if(o_bootable)
- {
- *o_bootable = false;
- break;
- }
- uint32_t mca_present = l_presMcaTargetList.size();
-
- /*@
- * @errortype
- * @severity ERRL_SEV_UNRECOVERABLE
- * @moduleid MOD_CHECK_MIN_HW
- * @reasoncode RC_SYSAVAIL_NO_MCAS_FUNC
- * @devdesc checkMinimumHardware found no
- * functional membufs
- * @custdesc A problem occurred during the IPL of the
- * system: Found no functional dimm cards.
- * @userdata1[00:31] HUID of node
- * @userdata2[00:31] number of present nonfunctional membufs
- */
- const uint64_t userdata1 =
- (static_cast<uint64_t>(get_huid(pTop)) << 32);
- const uint64_t userdata2 =
- (static_cast<uint64_t>(mca_present) << 32);
- l_errl = hwasError(ERRL_SEV_UNRECOVERABLE,
- MOD_CHECK_MIN_HW,
- RC_SYSAVAIL_NO_MCAS_FUNC,
- userdata1, userdata2);
-
- // call out the procedure to find the deconfigured part.
- hwasErrorAddProcedureCallout( l_errl,
- EPUB_PRC_FIND_DECONFIGURED_PART,
- SRCI_PRIORITY_HIGH );
-
- // if we already have an error, link this one to the earlier;
- // if not, set the common plid
- hwasErrorUpdatePlid( l_errl, l_commonPlid );
- errlCommit(l_errl, HWAS_COMP_ID);
- // errl is now NULL
- }
- }
- // there were no MCAs. If there are present membufs then
- // there must be some functional membufs
- else if (!l_presMembufTargetList.empty())
- {
- TargetHandleList l_funcMembufTargetList;
- PredicatePostfixExpr l_checkExprFunctionalMembufs;
- l_checkExprFunctionalMembufs.push(&l_membuf).
- push(&l_functional).And();
- targetService().getAssociated( l_funcMembufTargetList, pTop,
- TargetService::CHILD, TargetService::ALL,
- &l_checkExprFunctionalMembufs);
-
- HWAS_DBG( "checkMinimumHardware: %d functional membufs",
- l_funcMembufTargetList.size());
-
- if (l_funcMembufTargetList.empty())
- {
- HWAS_ERR( "Insufficient hardware to continue IPL"
- " (func membufs)");
- if(o_bootable)
- {
- *o_bootable = false;
- break;
- }
- TargetHandleList l_presentMembufTargetList;
- PredicatePostfixExpr l_checkExprPresentMembufs;
- l_checkExprPresentMembufs.push(&l_membuf).
- push(&l_present).And();
- targetService().getAssociated( l_presentMembufTargetList, pTop,
- TargetService::CHILD, TargetService::ALL,
- &l_checkExprPresentMembufs);
- uint32_t membufs_present = l_presentMembufTargetList.size();
-
- /*@
- * @errortype
- * @severity ERRL_SEV_UNRECOVERABLE
- * @moduleid MOD_CHECK_MIN_HW
- * @reasoncode RC_SYSAVAIL_NO_MEMBUFS_FUNC
- * @devdesc checkMinimumHardware found no
- * functional membufs
- * @custdesc A problem occurred during the IPL of the
- * system: Found no functional dimm cards.
- * @userdata1[00:31] HUID of node
- * @userdata2[00:31] number of present nonfunctional membufs
- */
- const uint64_t userdata1 =
- (static_cast<uint64_t>(get_huid(pTop)) << 32);
- const uint64_t userdata2 =
- (static_cast<uint64_t>(membufs_present) << 32);
- l_errl = hwasError(ERRL_SEV_UNRECOVERABLE,
- MOD_CHECK_MIN_HW,
- RC_SYSAVAIL_NO_MEMBUFS_FUNC,
- userdata1, userdata2);
-
- // call out the procedure to find the deconfigured part.
- hwasErrorAddProcedureCallout( l_errl,
- EPUB_PRC_FIND_DECONFIGURED_PART,
- SRCI_PRIORITY_HIGH );
-
- // if we already have an error, link this one to the earlier;
- // if not, set the common plid
- hwasErrorUpdatePlid( l_errl, l_commonPlid );
- errlCommit(l_errl, HWAS_COMP_ID);
- // errl is now NULL
- }
- }
- // No MCAs or membufs. If OMCBs are present, there must be some
- // functional OCMBs
- else if (!l_presOcmbTargetList.empty())
- {
-
- TargetHandleList l_funcOcmbTargetList;
- PredicatePostfixExpr l_checkExprFuncOcmbs;
- l_checkExprFuncOcmbs.push(&l_ocmb).push(&l_functional).And();
- targetService().getAssociated( l_funcOcmbTargetList, pTop,
- TargetService::CHILD, TargetService::ALL,
- &l_checkExprFuncOcmbs);
-
- HWAS_DBG( "checkMinimumHardware: %d functional OCMBs",
- l_funcOcmbTargetList.size());
-
- if (l_funcOcmbTargetList.empty())
- {
- HWAS_ERR( "Insufficient hardware to continue IPL"
- " (func OCMBs)");
- if(o_bootable)
- {
- *o_bootable = false;
- break;
- }
- TargetHandleList l_presentOcmbTargetList;
-
- PredicatePostfixExpr l_checkExprPresentOcmbs;
- l_checkExprPresentOcmbs.push(&l_ocmb).push(&l_present).And();
-
- targetService().getAssociated( l_presentOcmbTargetList, pTop,
- TargetService::CHILD, TargetService::ALL,
- &l_checkExprPresentOcmbs);
-
- uint32_t ocmbs_present = l_presentOcmbTargetList.size();
-
- /*@
- * @errortype
- * @severity ERRL_SEV_UNRECOVERABLE
- * @moduleid MOD_CHECK_MIN_HW
- * @reasoncode RC_SYSAVAIL_NO_OCMBS_FUNC
- * @devdesc checkMinimumHardware found no
- * functional ocmbs
- * @custdesc A problem occurred during the IPL of the
- * system: Found no functional dimm cards.
- * @userdata1[00:31] HUID of node
- * @userdata2[00:31] number of present nonfunctional ocmbs
- */
- const uint64_t userdata1 =
- (static_cast<uint64_t>(get_huid(pTop)) << 32);
- const uint64_t userdata2 =
- (static_cast<uint64_t>(ocmbs_present) << 32);
-
- l_errl = hwasError(ERRL_SEV_UNRECOVERABLE,
- MOD_CHECK_MIN_HW,
- RC_SYSAVAIL_NO_OCMBS_FUNC,
- userdata1, userdata2);
-
- // call out the procedure to find the deconfigured part.
- hwasErrorAddProcedureCallout( l_errl,
- EPUB_PRC_FIND_DECONFIGURED_PART,
- SRCI_PRIORITY_HIGH );
-
- // if we already have an error, link this one to the earlier;
- // if not, set the common plid
- hwasErrorUpdatePlid( l_errl, l_commonPlid );
- errlCommit(l_errl, HWAS_COMP_ID);
- // errl is now NULL
- }
- }
- else // No MCAs, membufs, or OCMB chips present. Cannot continue IPL.
- {
- HWAS_DBG("checkMinimumHardware: No present MCAs, membufs, "
- "or OCMBs.");
-
- HWAS_ERR( "Insufficient hardware to continue IPL"
- " (0 present MCAs, Membufs, OCMBs)");
-
- if(o_bootable)
- {
- *o_bootable = false;
- break;
- }
-
- /*@
- * @errortype
- * @severity ERRL_SEV_UNRECOVERABLE
- * @moduleid MOD_CHECK_MIN_HW
- * @reasoncode RC_SYSAVAIL_NO_MEMORY_PRESENT
- * @devdesc checkMinimumHardware found no present MCAs,
- * Membufs, or OCMB chips.
- * @custdesc A problem occurred during the IPL of the
- * system: Found no functional dimm cards.
- * @userdata1[00:31] HUID of node
- */
- const uint64_t userdata1 =
- (static_cast<uint64_t>(get_huid(pTop)) << 32);
-
- l_errl = hwasError(ERRL_SEV_UNRECOVERABLE,
- MOD_CHECK_MIN_HW,
- RC_SYSAVAIL_NO_MEMORY_PRESENT,
- userdata1);
-
- // call out the procedure to find the deconfigured part.
- hwasErrorAddProcedureCallout( l_errl,
- EPUB_PRC_FIND_DECONFIGURED_PART,
- SRCI_PRIORITY_HIGH );
-
- // if we already have an error, link this one to the earlier;
- // if not, set the common plid
- hwasErrorUpdatePlid( l_errl, l_commonPlid );
- errlCommit(l_errl, HWAS_COMP_ID);
- // errl is now NULL
- }
-
// check for functional NX chiplets
// Take specdeconfig into account here
TargetHandleList l_functionalNXChiplets;
OpenPOWER on IntegriCloud