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>
This commit is contained in:
Michiel W. van Tol
2018-07-24 15:28:34 +01:00
committed by Giacomo Travaglini
parent 50a57c0376
commit 2b3168021c
2 changed files with 8 additions and 4 deletions

View File

@@ -78,7 +78,6 @@ BaseTrafficGen::BaseTrafficGen(const BaseTrafficGenParams* p)
retryPkt(NULL),
retryPktTick(0),
updateEvent([this]{ update(); }, name()),
numSuppressed(0),
masterID(system->getMasterId(this))
{
}
@@ -186,9 +185,9 @@ BaseTrafficGen::update()
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 @@ BaseTrafficGen::regStats()
.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");

View File

@@ -163,7 +163,8 @@ class BaseTrafficGen : public MemObject
/** 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. */