diff options
author | Christian Geddes <crgeddes@us.ibm.com> | 2018-04-25 18:08:28 -0500 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2018-05-14 12:22:15 -0400 |
commit | b1771bfafee10eb48308457bab14f511394cf57d (patch) | |
tree | 53434237d5506920f4184d2ed8e6b0fbf704656d /src/import/chips/p9/procedures/xml | |
parent | d2f22373f239842f7eb9d9abbc3290c85894056b (diff) | |
download | talos-hostboot-b1771bfafee10eb48308457bab14f511394cf57d.tar.gz talos-hostboot-b1771bfafee10eb48308457bab14f511394cf57d.zip |
Create HWP to save/restore OBUS fir masks
During the IPL in a multi-node system Hostboot is unable to detect other
nodes. Because of this Hostboot is unable to set the PEER_TARGET attr
for OBUS targets as their PEERs are on different nodes. Once the system
reaching runtime HBRT will have the information to fill in PEER_TARGET
attributes on the OBUS targets. During the period while Hostboot doesn't
know the OBUS PEER_TARGETs we must mask off additional FIRs so that PRD
doesn't attempt to access the an OBUS's peer during a fail. This HWP will
provide a way to save off the FIR masks during IPL time, and will apply
additional masks required for the rest of the IPL. In addition this HWP
will provide a way to reapply the original masks after HBRT has filled
in the information for all OBUSes peer targets.
CQ: SW425530
Change-Id: I45a3d23023f0021d7dbf5c6726075b270e2e1418
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57853
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: Christopher W. Steffen <cwsteffen@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57857
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: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/xml')
-rw-r--r-- | src/import/chips/p9/procedures/xml/attribute_info/p9_io_obus_attributes.xml | 35 | ||||
-rwxr-xr-x | src/import/chips/p9/procedures/xml/error_info/p9_io_obus_errors.xml | 11 |
2 files changed, 44 insertions, 2 deletions
diff --git a/src/import/chips/p9/procedures/xml/attribute_info/p9_io_obus_attributes.xml b/src/import/chips/p9/procedures/xml/attribute_info/p9_io_obus_attributes.xml index f3467db81..a1f35c62f 100644 --- a/src/import/chips/p9/procedures/xml/attribute_info/p9_io_obus_attributes.xml +++ b/src/import/chips/p9/procedures/xml/attribute_info/p9_io_obus_attributes.xml @@ -115,6 +115,40 @@ </attribute> <!-- ********************************************************************** --> <attribute> + <id>ATTR_IO_OLLFIR_MASK</id> + <targetType>TARGET_TYPE_OBUS</targetType> + <description> + This attribute provides a place for Hostboot to save off the IO_OLLFIR_MASK. + During IPL time Hostboot wants to change the value of the FIR but it will + write the value back during Hostboot Runtime (HBRT). Hostboot will need to + wait to save the values until after p9_chiplet_scominit is called because that + HWP sets the fir masks we want to save off. Values will need to restored + after the PEER_TARGET values have been updated in HBRT + </description> + <valueType>uint64</valueType> + <initToZero/> + <writeable/> + <platInit/> +</attribute> +<!-- ********************************************************************** --> +<attribute> + <id>ATTR_IO_PB_IOOFIR_MASK</id> + <targetType>TARGET_TYPE_PROC_CHIP</targetType> + <description> + This attribute provides a place for Hostboot to save off the PB_IOOFIR_MASK. + During IPL time Hostboot wants to change the value of the FIR but it will + write the value back during Hostboot Runtime (HBRT). Hostboot will need to + wait to save the values until after p9_chiplet_scominit is called because that + HWP sets the fir masks we want to save off. Values will need to restored + after the PEER_TARGET values have been updated in HBRT + </description> + <valueType>uint64</valueType> + <initToZero/> + <writeable/> + <platInit/> +</attribute> +<!-- ********************************************************************** --> +<attribute> <id>ATTR_IO_O_CHANNEL_TYPE</id> <targetType>TARGET_TYPE_OBUS</targetType> <description> @@ -129,5 +163,4 @@ <platInit/> </attribute> <!-- ********************************************************************** --> - </attributes> diff --git a/src/import/chips/p9/procedures/xml/error_info/p9_io_obus_errors.xml b/src/import/chips/p9/procedures/xml/error_info/p9_io_obus_errors.xml index 19b0b8689..0efba8eeb 100755 --- a/src/import/chips/p9/procedures/xml/error_info/p9_io_obus_errors.xml +++ b/src/import/chips/p9/procedures/xml/error_info/p9_io_obus_errors.xml @@ -5,7 +5,7 @@ <!-- --> <!-- OpenPOWER HostBoot Project --> <!-- --> -<!-- Contributors Listed Below - COPYRIGHT 2015,2017 --> +<!-- Contributors Listed Below - COPYRIGHT 2015,2018 --> <!-- [+] International Business Machines Corp. --> <!-- --> <!-- --> @@ -53,4 +53,13 @@ </callout> </hwpError> <!-- *********************************************************************** --> + <hwpError> + <rc>RC_P9_IO_FIRMASK_RESTORE_ERROR</rc> + <description>Attempted to restore OBUS firmasks when values were never saved</description> + <ffdc>TARGET</ffdc> + <callout> + <procedure>CODE</procedure> + <priority>HIGH</priority> + </callout> + </hwpError> </hwpErrors> |