PRD: Fix skipping TPS ban after Maint UEs

Change-Id: Ib9c2d0506eed498038e1fc0a81d05494fa2a1b4f
CQ: SW409832
Backport: release-op910
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/50152
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com>
Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com>
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/50461
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>

1 files changed, 8 insertions, 8 deletions

diff --git a/src/usr/diag/prdf/plat/mem/prdfMemTps_rt.C b/src/usr/diag/prdf/plat/mem/prdfMemTps_rt.C
index 324ace8f9..20d835382 100644
--- a/src/usr/diag/prdf/plat/mem/prdfMemTps_rt.C
+++ b/src/usr/diag/prdf/plat/mem/prdfMemTps_rt.C
@@ -364,19 +364,19 @@ uint32_t TpsEvent<T>::analyzeTpsPhase1_rt( STEP_CODE_DATA_STRUCT & io_sc,
         // At this point, we are done with the procedure.
         o_done = true;
 
-        // If iv_ban is true, then ban TPS on this rank.
-        if ( iv_ban )
-        {
-            // It doesn't matter what we set the value to, we just need to
-            // make sure the rank exists in the map.
-            getMcaDataBundle(iv_chip)->iv_tpsBans[iv_rank] = true;
-        }
-
         // Since TPS is complete, clear the CE table for this slave rank.
         getMcaDataBundle(iv_chip)->iv_ceTable.deactivateRank( iv_rank );
 
     }while(0);
 
+    // If iv_ban is true and this procedure is done, then ban TPS on this rank.
+    if ( iv_ban && o_done )
+    {
+        // It doesn't matter what we set the value to, we just need to
+        // make sure the rank exists in the map.
+        getMcaDataBundle(iv_chip)->iv_tpsBans[iv_rank] = true;
+    }
+
     return o_rc;
 
     #undef PRDF_FUNC