summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Crowell <dcrowell@us.ibm.com>2019-12-02 10:53:13 -0600
committerWilliam G Hoffa <wghoffa@us.ibm.com>2019-12-17 14:41:22 -0600
commit75c0908b91275dc10bd17cb0f10b452f32ce0b91 (patch)
tree6b8c13470797c2cc193d0e247424c038fbfc1c83
parent0921b80a52a9cfd44186e0160d3dfae538746170 (diff)
downloadtalos-hostboot-75c0908b91275dc10bd17cb0f10b452f32ce0b91.tar.gz
talos-hostboot-75c0908b91275dc10bd17cb0f10b452f32ce0b91.zip
Switch DECONFIG over to DELAYED_DECONFIG
There is a race condition with the HWSV code on the FSP related to how deconfigurations are happened. A full solution on the FSP side would be difficult so we're making a few targeted fixes on the Hostboot side instead. Change-Id: I57a6ec80df77464131020499e43181c1869d2514 CQ: SW481625 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87928 Tested-by: Jenkins Server <pfd-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> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
-rw-r--r--src/usr/isteps/istep06/call_host_voltage_config.C6
-rw-r--r--src/usr/isteps/istep10/call_host_rng_bist.C2
-rw-r--r--src/usr/isteps/istep16/call_host_activate_slave_cores.C2
3 files changed, 5 insertions, 5 deletions
diff --git a/src/usr/isteps/istep06/call_host_voltage_config.C b/src/usr/isteps/istep06/call_host_voltage_config.C
index 6cb647e9b..5e7b0eb94 100644
--- a/src/usr/isteps/istep06/call_host_voltage_config.C
+++ b/src/usr/isteps/istep06/call_host_voltage_config.C
@@ -5,7 +5,7 @@
/* */
/* OpenPOWER HostBoot Project */
/* */
-/* Contributors Listed Below - COPYRIGHT 2016,2018 */
+/* Contributors Listed Below - COPYRIGHT 2016,2019 */
/* [+] International Business Machines Corp. */
/* */
/* */
@@ -489,7 +489,7 @@ void* call_host_voltage_config( void *io_pArgs )
l_err->addHwCallout(l_proc,
HWAS::SRCI_PRIORITY_HIGH,
- HWAS::DECONFIG,
+ HWAS::DELAYED_DECONFIG,
HWAS::GARD_NULL );
// Create IStep error log and
@@ -538,7 +538,7 @@ void* call_host_voltage_config( void *io_pArgs )
l_err->addHwCallout(l_proc,
HWAS::SRCI_PRIORITY_HIGH,
- HWAS::DECONFIG,
+ HWAS::DELAYED_DECONFIG,
HWAS::GARD_NULL );
// Create IStep error log and
diff --git a/src/usr/isteps/istep10/call_host_rng_bist.C b/src/usr/isteps/istep10/call_host_rng_bist.C
index e97476b02..70b82a274 100644
--- a/src/usr/isteps/istep10/call_host_rng_bist.C
+++ b/src/usr/isteps/istep10/call_host_rng_bist.C
@@ -120,7 +120,7 @@ void* call_host_rng_bist( void *io_pArgs )
{
l_err->addHwCallout( l_nxTarget,
HWAS::SRCI_PRIORITY_HIGH,
- HWAS::DECONFIG,
+ HWAS::DELAYED_DECONFIG,
HWAS::GARD_NULL );
}
}
diff --git a/src/usr/isteps/istep16/call_host_activate_slave_cores.C b/src/usr/isteps/istep16/call_host_activate_slave_cores.C
index c5f941e8f..1abc215ed 100644
--- a/src/usr/isteps/istep16/call_host_activate_slave_cores.C
+++ b/src/usr/isteps/istep16/call_host_activate_slave_cores.C
@@ -293,7 +293,7 @@ void* call_host_activate_slave_cores (void *io_pArgs)
// Callout and gard core that failed to wake up.
l_errl->addHwCallout(*l_core,
HWAS::SRCI_PRIORITY_HIGH,
- HWAS::DECONFIG,
+ HWAS::DELAYED_DECONFIG,
HWAS::GARD_Predictive);
// Could be an interrupt issue
OpenPOWER on IntegriCloud