PRD: capture NPU FIRs only on attention from NPU FIRs

Change-Id: I9576aa1205baeabc40eac7c20359d95e9d1d8758
CQ: SW424468
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57214
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: Caleb N. Palmer <cnpalmer@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57332
CI-Ready: Zane C. Shelley <zshelle@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Zane C. Shelley <zshelle@us.ibm.com>

1 files changed, 13 insertions, 1 deletions

diff --git a/src/usr/diag/prdf/common/framework/config/iipSystem.C b/src/usr/diag/prdf/common/framework/config/iipSystem.C
index 087347b93..4e3f04893 100755
--- a/src/usr/diag/prdf/common/framework/config/iipSystem.C
+++ b/src/usr/diag/prdf/common/framework/config/iipSystem.C
@@ -5,7 +5,7 @@
 /*                                                                        */
 /* OpenPOWER HostBoot Project                                             */
 /*                                                                        */
-/* Contributors Listed Below - COPYRIGHT 2012,2017                        */
+/* Contributors Listed Below - COPYRIGHT 2012,2018                        */
 /* [+] International Business Machines Corp.                              */
 /*                                                                        */
 /*                                                                        */
@@ -353,6 +353,18 @@ int32_t System::Analyze(STEP_CODE_DATA_STRUCT & serviceData,
                                                     *serviceData.service_data );
                         // Set up Error Isolation Pass Flag.
                         serviceData.service_data->setIsolationOnlyPass();
+                        // The original capture data is held with l_temp_sdc
+                        // when the copy contructor was called above. If we
+                        // continue to use serviceData.service_data as is, it
+                        // still contains all of the current capture data. So
+                        // any additional capture data will most likely be
+                        // duplicates (which will actually be a third copy of
+                        // the registers). The duplicates will be removed
+                        // eventually when mergeData() is called later, but we
+                        // can prevent a lot of duplication and have a improve
+                        // preformance slightly by clearing the capture data for
+                        // the isolation pass.
+                        serviceData.service_data->GetCaptureData().Clear();
                         // Set the outer for loop iteration variable atnType so
                         // that we analyze MACHINE XSTOP in next iteration.
                         atnType = MACHINE_CHECK + 1;