Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/51674 )
(
1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the
submitted one.
)Change subject: cpu,tests: Replace the deprecated Stats namespace with
statistics.
......................................................................
cpu,tests: Replace the deprecated Stats namespace with statistics.
The gups traffic generator was checked in recently and uses the
deprecated namespace. This change updates it.
Change-Id: I5e6f593aac086e6ef251f11fc50c0e3d3e545d06
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/51674
Maintainer: Gabe Black <[email protected]>
Reviewed-by: Daniel Carvalho <[email protected]>
Tested-by: kokoro <[email protected]>
---
M src/cpu/testers/traffic_gen/gups_gen.cc
M src/cpu/testers/traffic_gen/gups_gen.hh
2 files changed, 31 insertions(+), 15 deletions(-)
Approvals:
Daniel Carvalho: Looks good to me, approved
Gabe Black: 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 615d4b0..3ed1fa4 100644
--- a/src/cpu/testers/traffic_gen/gups_gen.cc
+++ b/src/cpu/testers/traffic_gen/gups_gen.cc
@@ -324,8 +324,8 @@
return true;
}
-GUPSGen::GUPSGenStat::GUPSGenStat(GUPSGen* parent):
- Stats::Group(parent),
+GUPSGen::GUPSGenStat::GUPSGenStat(GUPSGen* parent) :
+ statistics::Group(parent),
ADD_STAT(totalUpdates, statistics::units::Count::get(),
"Total number of updates the generator made in the memory"),
ADD_STAT(GUPS, statistics::units::Rate<statistics::units::Count,
diff --git a/src/cpu/testers/traffic_gen/gups_gen.hh
b/src/cpu/testers/traffic_gen/gups_gen.hh
index 04c0bb0..fa7b336 100644
--- a/src/cpu/testers/traffic_gen/gups_gen.hh
+++ b/src/cpu/testers/traffic_gen/gups_gen.hh
@@ -301,25 +301,25 @@
*/
int readRequests;
- struct GUPSGenStat : public Stats::Group
+ struct GUPSGenStat : public statistics::Group
{
GUPSGenStat(GUPSGen* parent);
void regStats() override;
- Stats::Scalar totalUpdates;
- Stats::Formula GUPS;
+ statistics::Scalar totalUpdates;
+ statistics::Formula GUPS;
- Stats::Scalar totalReads;
- Stats::Scalar totalBytesRead;
- Stats::Formula avgReadBW;
- Stats::Scalar totalReadLat;
- Stats::Formula avgReadLat;
+ statistics::Scalar totalReads;
+ statistics::Scalar totalBytesRead;
+ statistics::Formula avgReadBW;
+ statistics::Scalar totalReadLat;
+ statistics::Formula avgReadLat;
- Stats::Scalar totalWrites;
- Stats::Scalar totalBytesWritten;
- Stats::Formula avgWriteBW;
- Stats::Scalar totalWriteLat;
- Stats::Formula avgWriteLat;
+ statistics::Scalar totalWrites;
+ statistics::Scalar totalBytesWritten;
+ statistics::Formula avgWriteBW;
+ statistics::Scalar totalWriteLat;
+ statistics::Formula avgWriteLat;
} stats;
public:
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/51674
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: I5e6f593aac086e6ef251f11fc50c0e3d3e545d06
Gerrit-Change-Number: 51674
Gerrit-PatchSet: 3
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Mahyar Samani <[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
