From b05b7eb24e29eff4a767a7e0d4883a16a989a55b Mon Sep 17 00:00:00 2001
From: Prem Shanker Jha <premjha2@in.ibm.com>
Date: Fri, 13 Sep 2019 00:10:41 -0500
Subject: PM: Modified FFDC to avoid corruption of RC in error path.

On FSP platform, in case of some errors, HWP captures
certain EX level registers. To ensure availability of
such registers, FSP first attempts special wakeup.
Special wakeup HWP uses FAPI_TRY in its code path. It
is known that in FFDC collection path, use of FAPI_TRY
leads to corruption of original FAPI2 RC. Commit
removes such registers from the capture list.

Key_Cronus_Test=PM_REGRESS

Change-Id: I23ce8b1a8a4a61f71c7b33c1122fcfac905616c4
CQ: SW471606
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83863
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Gregory S Still <stillgs@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: Cronus HW CI <cronushw-ci+hostboot@us.ibm.com>
Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: AMIT J. TENDOLKAR <amit.tendolkar@in.ibm.com>
Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83956
Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
---
 .../xml/error_info/p9_sbe_check_master_stop15_errors.xml | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

(limited to 'src/import/chips/p9/procedures')

diff --git a/src/import/chips/p9/procedures/xml/error_info/p9_sbe_check_master_stop15_errors.xml b/src/import/chips/p9/procedures/xml/error_info/p9_sbe_check_master_stop15_errors.xml
index 15b26599..181f5cbc 100644
--- a/src/import/chips/p9/procedures/xml/error_info/p9_sbe_check_master_stop15_errors.xml
+++ b/src/import/chips/p9/procedures/xml/error_info/p9_sbe_check_master_stop15_errors.xml
@@ -5,7 +5,7 @@
 <!--                                                                        -->
 <!-- OpenPOWER sbe Project                                                  -->
 <!--                                                                        -->
-<!-- Contributors Listed Below - COPYRIGHT 2016,2018                        -->
+<!-- Contributors Listed Below - COPYRIGHT 2016,2019                        -->
 <!-- [+] International Business Machines Corp.                              -->
 <!--                                                                        -->
 <!--                                                                        -->
@@ -34,6 +34,8 @@
       for a completed transition.
     </description>
 
+    <ffdc>EX</ffdc>
+
     <collectFfdc>p9_eq_clear_atomic_lock,EQ</collectFfdc>
 
     <collectRegisterFfdc>
@@ -46,11 +48,6 @@
         <target>EQ</target>
         <targetType>TARGET_TYPE_EQ</targetType>
     </collectRegisterFfdc>
-    <collectRegisterFfdc>
-        <id>CHECK_MASTER_STOP15_FFDC_REGS_EX</id>
-        <target>EX</target>
-        <targetType>TARGET_TYPE_EX</targetType>
-    </collectRegisterFfdc>
     <collectRegisterFfdc>
         <id>CHECK_MASTER_STOP15_FFDC_REGS_EC</id>
         <target>EC</target>
@@ -77,6 +74,8 @@
       but the achieved level is not appropriate.
     </description>
 
+    <ffdc>EX</ffdc>
+
     <collectFfdc>p9_eq_clear_atomic_lock,EQ</collectFfdc>
 
     <collectRegisterFfdc>
@@ -89,11 +88,6 @@
         <target>EQ</target>
         <targetType>TARGET_TYPE_EQ</targetType>
     </collectRegisterFfdc>
-    <collectRegisterFfdc>
-        <id>CHECK_MASTER_STOP15_FFDC_REGS_EX</id>
-        <target>EX</target>
-        <targetType>TARGET_TYPE_EX</targetType>
-    </collectRegisterFfdc>
     <collectRegisterFfdc>
         <id>CHECK_MASTER_STOP15_FFDC_REGS_EC</id>
         <target>EC</target>
-- 
cgit v1.2.1