HTMGT: WOF Reset Disable Flag

Change-Id: Ia4c8b0342b668a150f1f79c49f9c7bba2614ca34 RTC:192844 CQ: SW439350 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: Christopher J. Cain <cjcain@us.ibm.com> Reviewed-by: Sheldon Bailey <baileysh@us.ibm.com> Reviewed-by: Martha Broyles <mbroyles@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63227 Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
open-power · Jul 27, 2018 · 52d75b9 · 52d75b9
1 parent 58b3d49
commit 52d75b9
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 10 deletions.
diff --git a/src/usr/htmgt/htmgt_cfgdata.C b/src/usr/htmgt/htmgt_cfgdata.C
@@ -1579,8 +1579,10 @@ bool check_wof_support(uint16_t & o_nominal,
                 }
                 else if( largest_wof_reset_count >= WOF_RESET_COUNT_THRESHOLD )
                 {
-                    TMGT_INF("WOF reset count reached for OCC%d. WOF disabled.",
-                             occ_instance);
+                    TMGT_INF("WOF reset count reached for "
+                         "OCC%d count: %d. WOF disabled.",
+                         occ_instance,
+                         largest_wof_reset_count );
                     G_wofSupported = false;
                     o_ultra = WOF_RESET_COUNT_REACHED;
                 }

diff --git a/src/usr/htmgt/htmgt_occ.C b/src/usr/htmgt/htmgt_occ.C
@@ -193,8 +193,13 @@ namespace HTMGT
         uint8_t cmdData[2];
         cmdData[0] = OCC_RESET_CMD_VERSION;
 
-        TMGT_INF("resetPrep: OCC%d (failed=%c, reset count=%d)",
-                 iv_instance, iv_failed?'y':'n', iv_resetCount);
+        TMGT_INF("resetPrep: OCC%d (failed=%c, reset count=%d)"
+                 " reset reason=0x%x",
+                 iv_instance,
+                 iv_failed?'y':'n',
+                 iv_resetCount,
+                 iv_resetReason);
+
         if(iv_failed)
         {
             cmdData[1] = OCC_RESET_FAIL_THIS_OCC;

diff --git a/src/usr/htmgt/htmgt_utility.H b/src/usr/htmgt/htmgt_utility.H
@@ -131,6 +131,7 @@ namespace HTMGT
     enum internalFlagTypes
     {
         FLAG_HALT_ON_OCC_SRC       = 0x00800000, // Prevent resets after SRC
+        FLAG_WOF_RESET_DISABLED    = 0x00001000, // Disable ONLY WOF resets
         FLAG_DISABLE_MEM_CONFIG    = 0x00000800, // Disable sending mem config
         FLAG_HALT_ON_RESET_FAIL    = 0x00000100, // Stop additional resets
         FLAG_EXT_RESET_DISABLED    = 0x00000080, // Ignore opal-prd/BMC resets

diff --git a/src/usr/htmgt/occError.C b/src/usr/htmgt/occError.C
@@ -419,20 +419,30 @@ namespace HTMGT
 
     } // end Occ::elogAddCallout()
 
-
     void Occ::elogProcessActions(const uint8_t i_actions,
                                  bool        & o_occReset,
                                  ERRORLOG::errlSeverity_t & o_errlSeverity)
     {
         if (i_actions & TMGT_ERRL_ACTIONS_WOF_RESET_REQUIRED)
         {
-            o_occReset = true;
             iv_failed = false;
-            iv_needsWofReset = true;
             iv_resetReason = OCC_RESET_REASON_WOF_REQUEST;
-
-            TMGT_INF("elogProcessActions: OCC%d requested a WOF reset",
-                     iv_instance);
+            // Check if WOF resets are disabled
+            if(int_flags_set(FLAG_WOF_RESET_DISABLED) == true)
+            {
+                o_occReset = false;
+                iv_needsWofReset = false;
+                TMGT_INF("elogProcessActions: OCC%d requested a WOF reset "
+                         "but WOF resets are DISABLED",
+                         iv_instance);
+            }
+            else // WOF resets are enabled
+            {
+                o_occReset = true;
+                iv_needsWofReset = true;
+                TMGT_INF("elogProcessActions: OCC%d requested a WOF reset",
+                         iv_instance);
+            }
         }
         else
         {