diff options
author | Christian Geddes <crgeddes@us.ibm.com> | 2018-05-02 17:03:14 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-05-14 11:31:57 -0400 |
commit | d2f22373f239842f7eb9d9abbc3290c85894056b (patch) | |
tree | 03ba4394a61514e0ce236f614fd961db445e6396 /src/usr | |
parent | 927220a65bb74c95e7902158ec2bc6642c596531 (diff) | |
download | talos-hostboot-d2f22373f239842f7eb9d9abbc3290c85894056b.tar.gz talos-hostboot-d2f22373f239842f7eb9d9abbc3290c85894056b.zip |
Deconfig record can't get added to vitalAttn elog on FSP
On FSP system we should never try to call vitalAttnHandler because
the fir bits that trigger this code should be masked. However
we have seen issues where for whatever reason the firs get unmasked.
If we happen to fall into this code on a FSP system we want to make
sure that no deconfigure records get added to the error logs that
get generated. If deconfig records get added it can collide with
the flow that hwsv is doing to attempt to recover the SBE. Also
in this commit we removed the TODO for disabling the OCC if the SBE
dies. We have decided that it is no longer necessary to disable
the OCC if the SBE dies. See story for more information.
Change-Id: Ib75dc34a122a37853047af67ec8858dbca11e4a2
CQ: SW427127
RTC: 180244
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/58226
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Reviewed-by: Brian E. Bakke <bbakke@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>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Roland Veloz <rveloz@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Diffstat (limited to 'src/usr')
-rw-r--r-- | src/usr/sbeio/common/sbe_attn.C | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/src/usr/sbeio/common/sbe_attn.C b/src/usr/sbeio/common/sbe_attn.C index 5d8be78e4..e6adafc2b 100644 --- a/src/usr/sbeio/common/sbe_attn.C +++ b/src/usr/sbeio/common/sbe_attn.C @@ -114,7 +114,6 @@ namespace SBEIO HWAS::GARD_NULL ); l_err->collectTrace( SBEIO_COMP_NAME, 256); - // @todo - RTC:180244 - Disable the OCC } // Inform OPAL the state of the SBE after a retry is successful else @@ -164,12 +163,23 @@ namespace SBEIO TARGETING::get_huid(i_procTarg), l_ret); } - - //We want to deconfigure the processor where the error was detected +// If this code happens to get called on a FSP system during runtime we definitely do not +// want to add a deconfig record. If we do it will cause conflict with HWSV code which might +// be trying to handle the SBE error +#ifndef CONFIG_FSP_BUILD + // We want to deconfigure the processor where the error was detected l_err->addHwCallout( i_procTarg, HWAS::SRCI_PRIORITY_HIGH, HWAS::DELAYED_DECONFIG, HWAS::GARD_NULL ); +#else + // We want to only add a hw callout for the processor where the error was detected + // if we fall down this path on an FSP system. + l_err->addHwCallout( i_procTarg, + HWAS::SRCI_PRIORITY_HIGH, + HWAS::NO_DECONFIG, + HWAS::GARD_NULL ); +#endif #endif |