Mahyar Samani has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/51608 )

Change subject: cpu: Updating stats for GUPSGen
......................................................................

cpu: Updating stats for GUPSGen

This changes deletes the deprecated stat types for GUPSGen and
replaces them with the namespaces.

Change-Id: I3a6f1bd65c0b5ebad613b8769553942d9b0d8e0c
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/51608
Reviewed-by: Jason Lowe-Power <power...@gmail.com>
Maintainer: Jason Lowe-Power <power...@gmail.com>
Tested-by: kokoro <noreply+kok...@google.com>
---
M src/cpu/testers/traffic_gen/gups_gen.cc
1 file changed, 31 insertions(+), 12 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass




diff --git a/src/cpu/testers/traffic_gen/gups_gen.cc b/src/cpu/testers/traffic_gen/gups_gen.cc
index 837b234..615d4b0 100644
--- a/src/cpu/testers/traffic_gen/gups_gen.cc
+++ b/src/cpu/testers/traffic_gen/gups_gen.cc
@@ -326,29 +326,32 @@

 GUPSGen::GUPSGenStat::GUPSGenStat(GUPSGen* parent):
     Stats::Group(parent),
-    ADD_STAT(totalUpdates, UNIT_COUNT,
+    ADD_STAT(totalUpdates, statistics::units::Count::get(),
         "Total number of updates the generator made in the memory"),
-    ADD_STAT(GUPS, UNIT_RATE(Stats::Units::Count, Stats::Units::Second),
+    ADD_STAT(GUPS, statistics::units::Rate<statistics::units::Count,
+                statistics::units::Second>::get(),
         "Rate of billion updates per second"),
-    ADD_STAT(totalReads, UNIT_COUNT,
+    ADD_STAT(totalReads, statistics::units::Count::get(),
         "Total number of read requests"),
-    ADD_STAT(totalBytesRead, UNIT_BYTE,
+    ADD_STAT(totalBytesRead, statistics::units::Byte::get(),
         "Total number of bytes read"),
- ADD_STAT(avgReadBW, UNIT_RATE(Stats::Units::Byte, Stats::Units::Second),
+    ADD_STAT(avgReadBW, statistics::units::Rate<statistics::units::Byte,
+                statistics::units::Second>::get(),
         "Average read bandwidth received from memory"),
-    ADD_STAT(totalReadLat, UNIT_TICK,
+    ADD_STAT(totalReadLat, statistics::units::Tick::get(),
         "Total latency of read requests."),
-    ADD_STAT(avgReadLat, UNIT_TICK,
+    ADD_STAT(avgReadLat, statistics::units::Tick::get(),
         "Average latency for read requests"),
-    ADD_STAT(totalWrites, UNIT_COUNT,
+    ADD_STAT(totalWrites, statistics::units::Count::get(),
         "Total number of write requests"),
-    ADD_STAT(totalBytesWritten, UNIT_BYTE,
+    ADD_STAT(totalBytesWritten, statistics::units::Byte::get(),
         "Total number of bytes written"),
- ADD_STAT(avgWriteBW, UNIT_RATE(Stats::Units::Byte, Stats::Units::Second),
+    ADD_STAT(avgWriteBW, statistics::units::Rate<statistics::units::Byte,
+                statistics::units::Second>::get(),
         "Average write bandwidth received from memory"),
-    ADD_STAT(totalWriteLat, UNIT_TICK,
+    ADD_STAT(totalWriteLat, statistics::units::Tick::get(),
         "Total latency of write requests."),
-    ADD_STAT(avgWriteLat, UNIT_TICK,
+    ADD_STAT(avgWriteLat, statistics::units::Tick::get(),
         "Average latency for write requests")
 {}


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/51608
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: I3a6f1bd65c0b5ebad613b8769553942d9b0d8e0c
Gerrit-Change-Number: 51608
Gerrit-PatchSet: 2
Gerrit-Owner: Mahyar Samani <msam...@ucdavis.edu>
Gerrit-Reviewer: Gabe Black <gabe.bl...@gmail.com>
Gerrit-Reviewer: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: Mahyar Samani <msam...@ucdavis.edu>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
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