Hoa Nguyen has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/39975 )

Change subject: mem: Fix/Improve stats in src/mem
......................................................................

mem: Fix/Improve stats in src/mem

* Improved the stat description of `occupancies` stat of cache tags.
* Fixed the stat description of `ageTaskId` stat of cache tags.
* Fixed the value assigned to `warmupCycle` stat of cache tags,
it was set to curTick().

Change-Id: Iabff7d42318fcc73df79ee7f3c5430f4c45555e5
Signed-off-by: Hoa Nguyen <hoangu...@ucdavis.edu>
---
M src/mem/cache/tags/base.cc
1 file changed, 3 insertions(+), 3 deletions(-)



diff --git a/src/mem/cache/tags/base.cc b/src/mem/cache/tags/base.cc
index b12fdf0..8b1699c 100644
--- a/src/mem/cache/tags/base.cc
+++ b/src/mem/cache/tags/base.cc
@@ -115,7 +115,7 @@
     // Check if cache warm up is done
     if (!warmedUp && stats.tagsInUse.value() >= warmupBound) {
         warmedUp = true;
-        stats.warmupCycle = curTick();
+        stats.warmupCycle = ticksToCycles(curTick());
     }

     // We only need to write into one tag and one data block.
@@ -221,10 +221,10 @@
     ADD_STAT(sampledRefs, "Sample count of references to valid blocks."),
     ADD_STAT(avgRefs, "Average number of references to valid blocks."),
     ADD_STAT(warmupCycle, "Cycle when the warmup percentage was hit."),
-    ADD_STAT(occupancies, "Average occupied blocks per requestor"),
+ ADD_STAT(occupancies, "Average occupied blocks per tick, per requestor"),
     ADD_STAT(avgOccs, "Average percentage of cache occupancy"),
     ADD_STAT(occupanciesTaskId, "Occupied blocks per task id"),
-    ADD_STAT(ageTaskId, "Occupied blocks per task id"),
+    ADD_STAT(ageTaskId, "Occupied blocks per task id, per block age"),
ADD_STAT(percentOccsTaskId, "Percentage of cache occupancy per task id"),
     ADD_STAT(tagAccesses, "Number of tag accesses"),
     ADD_STAT(dataAccesses, "Number of data accesses")

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/39975
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: Iabff7d42318fcc73df79ee7f3c5430f4c45555e5
Gerrit-Change-Number: 39975
Gerrit-PatchSet: 1
Gerrit-Owner: Hoa Nguyen <hoangu...@ucdavis.edu>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to