commit 2b3168021cb0ceed8e90045435181bb82eed0c12
author Michiel W. van Tol <Michiel.VanTol@arm.com> Tue Jul 24 15:28:34 2018 +0100
committer Giacomo Travaglini <giacomo.travaglini@arm.com> Fri Aug 24 13:23:40 2018 +0000
tree d88d7c87e70a15d46798316e53d12e10ff42c16f
parent 50a57c0376c02c912a978c4443dd58caebe0f173

cpu: Turn BaseTrafficGen numSuppressed into a stat

This is changing numSuppressed from being a warn only variable into
a Stat so that it is visible at the end of simulation.

Change-Id: I934782e796c898bfc0e773cc88c597a68e403272
Reviewed-on: https://gem5-review.googlesource.com/11849
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>

src/cpu/testers/traffic_gen/base.cc

diff --git a/src/cpu/testers/traffic_gen/base.cc b/src/cpu/testers/traffic_gen/base.cc
index 41d0052..6778e41 100644
--- a/src/cpu/testers/traffic_gen/base.cc
+++ b/src/cpu/testers/traffic_gen/base.cc
@@ -78,7 +78,6 @@
       retryPkt(NULL),
       retryPktTick(0),
       updateEvent([this]{ update(); }, name()),
-      numSuppressed(0),
       masterID(system->getMasterId(this))
 {
 }
@@ -186,9 +185,9 @@
                     pkt->cmdString(), pkt->getAddr());
 
             ++numSuppressed;
-            if (numSuppressed % 10000)
+            if (!(static_cast<int>(numSuppressed.value()) % 10000))
                 warn("%s suppressed %d packets with non-memory addresses\n",
-                     name(), numSuppressed);
+                     name(), numSuppressed.value());
 
             delete pkt;
             pkt = nullptr;
@@ -314,6 +313,10 @@
         .name(name() + ".numPackets")
         .desc("Number of packets generated");
 
+    numSuppressed
+        .name(name() + ".numSuppressed")
+        .desc("Number of suppressed packets to non-memory space");
+
     numRetries
         .name(name() + ".numRetries")
         .desc("Number of retries");

src/cpu/testers/traffic_gen/base.hh

diff --git a/src/cpu/testers/traffic_gen/base.hh b/src/cpu/testers/traffic_gen/base.hh
index 080bd78..fe4229f 100644
--- a/src/cpu/testers/traffic_gen/base.hh
+++ b/src/cpu/testers/traffic_gen/base.hh
@@ -163,7 +163,8 @@
     /** Event for scheduling updates */
     EventFunctionWrapper updateEvent;
 
-    uint64_t numSuppressed;
+    /** Count the number of dropped requests. */
+    Stats::Scalar numSuppressed;
 
   private: // Stats
     /** Count the number of generated packets. */