Huang Jiasen has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/53103 )
Change subject: mem: Setup stats for Cache hitLatency
......................................................................
mem: Setup stats for Cache hitLatency
Change-Id: I18de75d3b6c4ce1784b90653e2d132ffecf1b1af
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/53103
Tested-by: kokoro <[email protected]>
Reviewed-by: Jason Lowe-Power <[email protected]>
Maintainer: Jason Lowe-Power <[email protected]>
---
M src/mem/cache/base.cc
M src/mem/cache/base.hh
2 files changed, 48 insertions(+), 0 deletions(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/src/mem/cache/base.cc b/src/mem/cache/base.cc
index f97c30a..dc21151 100644
--- a/src/mem/cache/base.cc
+++ b/src/mem/cache/base.cc
@@ -1954,6 +1954,8 @@
("number of " + name + " hits").c_str()),
ADD_STAT(misses, statistics::units::Count::get(),
("number of " + name + " misses").c_str()),
+ ADD_STAT(hitLatency, statistics::units::Tick::get(),
+ ("number of " + name + " hit ticks").c_str()),
ADD_STAT(missLatency, statistics::units::Tick::get(),
("number of " + name + " miss ticks").c_str()),
ADD_STAT(accesses, statistics::units::Count::get(),
@@ -2010,6 +2012,15 @@
misses.subname(i, system->getRequestorName(i));
}
+ // Hit latency statistics
+ hitLatency
+ .init(max_requestors)
+ .flags(total | nozero | nonan)
+ ;
+ for (int i = 0; i < max_requestors; i++) {
+ hitLatency.subname(i, system->getRequestorName(i));
+ }
+
// Miss latency statistics
missLatency
.init(max_requestors)
@@ -2116,6 +2127,10 @@
"number of demand (read+write) hits"),
ADD_STAT(overallHits, statistics::units::Count::get(),
"number of overall hits"),
+ ADD_STAT(demandHitLatency, statistics::units::Tick::get(),
+ "number of demand (read+write) hit ticks"),
+ ADD_STAT(overallHitLatency, statistics::units::Tick::get(),
+ "number of overall hit ticks"),
ADD_STAT(demandMisses, statistics::units::Count::get(),
"number of demand (read+write) misses"),
ADD_STAT(overallMisses, statistics::units::Count::get(),
@@ -2250,6 +2265,17 @@
overallMissLatency.subname(i, system->getRequestorName(i));
}
+ demandHitLatency.flags(total | nozero | nonan);
+ demandHitLatency = SUM_DEMAND(hitLatency);
+ for (int i = 0; i < max_requestors; i++) {
+ demandHitLatency.subname(i, system->getRequestorName(i));
+ }
+ overallHitLatency.flags(total | nozero | nonan);
+ overallHitLatency = demandHitLatency + SUM_NON_DEMAND(hitLatency);
+ for (int i = 0; i < max_requestors; i++) {
+ overallHitLatency.subname(i, system->getRequestorName(i));
+ }
+
demandAccesses.flags(total | nozero | nonan);
demandAccesses = demandHits + demandMisses;
for (int i = 0; i < max_requestors; i++) {
diff --git a/src/mem/cache/base.hh b/src/mem/cache/base.hh
index 988a678..0dc64e1 100644
--- a/src/mem/cache/base.hh
+++ b/src/mem/cache/base.hh
@@ -1006,6 +1006,11 @@
@sa Packet::Command */
statistics::Vector misses;
/**
+ * Total number of cycles per thread/command spent waiting for a
hit.
+ * Used to calculate the average hit latency.
+ */
+ statistics::Vector hitLatency;
+ /**
* Total number of cycles per thread/command spent waiting for a
miss.
* Used to calculate the average miss latency.
*/
@@ -1050,6 +1055,10 @@
statistics::Formula demandHits;
/** Number of hit for all accesses. */
statistics::Formula overallHits;
+ /** Total number of cycles spent waiting for demand hits. */
+ statistics::Formula demandHitLatency;
+ /** Total number of cycles spent waiting for all hits. */
+ statistics::Formula overallHitLatency;
/** Number of misses for demand accesses. */
statistics::Formula demandMisses;
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/53103
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I18de75d3b6c4ce1784b90653e2d132ffecf1b1af
Gerrit-Change-Number: 53103
Gerrit-PatchSet: 13
Gerrit-Owner: Huang Jiasen <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Huang Jiasen <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Nikos Nikoleris <[email protected]>
Gerrit-Reviewer: Tom Rollet <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
