Emily Brickey has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/32835 )
Change subject: arch-riscv, arch-x86: convert tlb to new style stats
......................................................................
arch-riscv, arch-x86: convert tlb to new style stats
Change-Id: Ie2754d861a658fde0acdda30cbcb91e02029e33a
---
M src/arch/riscv/tlb.cc
M src/arch/riscv/tlb.hh
M src/arch/x86/tlb.cc
M src/arch/x86/tlb.hh
4 files changed, 57 insertions(+), 109 deletions(-)
diff --git a/src/arch/riscv/tlb.cc b/src/arch/riscv/tlb.cc
index c8b1a4c..b74c1a1 100644
--- a/src/arch/riscv/tlb.cc
+++ b/src/arch/riscv/tlb.cc
@@ -66,7 +66,7 @@
}
TLB::TLB(const Params *p)
- : BaseTLB(p), size(p->size), tlb(size), lruSeq(0)
+ : BaseTLB(p), size(p->size), tlb(size), lruSeq(0), stats(this)
{
for (size_t x = 0; x < size; x++) {
tlb[x].trieHandle = NULL;
@@ -108,21 +108,21 @@
entry->lruSeq = nextSeq();
if (mode == Write)
- write_accesses++;
+ stats.write_accesses++;
else
- read_accesses++;
+ stats.read_accesses++;
if (!entry) {
if (mode == Write)
- write_misses++;
+ stats.write_misses++;
else
- read_misses++;
+ stats.read_misses++;
}
else {
if (mode == Write)
- write_hits++;
+ stats.write_hits++;
else
- read_hits++;
+ stats.read_hits++;
}
DPRINTF(TLBVerbose, "lookup(vpn=%#x, asid=%#x): %s ppn %#x\n",
@@ -496,61 +496,20 @@
}
}
-void
-TLB::regStats()
+TLB::TlbStats::TlbStats(Stats::Group *parent)
+ : Stats::Group(parent),
+ ADD_STAT(read_hits, "DTB read hits"),
+ ADD_STAT(read_misses, "DTB read misses"),
+ ADD_STAT(read_accesses, "DTB read accesses"),
+ ADD_STAT(write_hits, "DTB write hits"),
+ ADD_STAT(write_misses, "DTB write misses"),
+ ADD_STAT(write_accesses, "DTB write accesses"),
+ ADD_STAT(hits, "Total TLB (read and write) hits", read_hits +
write_hits),
+ ADD_STAT(misses, "Total TLB (read and write) misses",
+ read_misses + write_misses),
+ ADD_STAT(accesses, "Total TLB (read and write) accesses",
+ read_accesses + write_accesses)
{
- BaseTLB::regStats();
-
- read_hits
- .name(name() + ".read_hits")
- .desc("DTB read hits")
- ;
-
- read_misses
- .name(name() + ".read_misses")
- .desc("DTB read misses")
- ;
-
-
- read_accesses
- .name(name() + ".read_accesses")
- .desc("DTB read accesses")
- ;
-
- write_hits
- .name(name() + ".write_hits")
- .desc("DTB write hits")
- ;
-
- write_misses
- .name(name() + ".write_misses")
- .desc("DTB write misses")
- ;
-
-
- write_accesses
- .name(name() + ".write_accesses")
- .desc("DTB write accesses")
- ;
-
- hits
- .name(name() + ".hits")
- .desc("DTB hits")
- ;
-
- misses
- .name(name() + ".misses")
- .desc("DTB misses")
- ;
-
- accesses
- .name(name() + ".accesses")
- .desc("DTB accesses")
- ;
-
- hits = read_hits + write_hits;
- misses = read_misses + write_misses;
- accesses = read_accesses + write_accesses;
}
RiscvISA::TLB *
diff --git a/src/arch/riscv/tlb.hh b/src/arch/riscv/tlb.hh
index 7ed6628..7918d0e 100644
--- a/src/arch/riscv/tlb.hh
+++ b/src/arch/riscv/tlb.hh
@@ -64,17 +64,23 @@
Walker *walker;
- mutable Stats::Scalar read_hits;
- mutable Stats::Scalar read_misses;
- mutable Stats::Scalar read_acv;
- mutable Stats::Scalar read_accesses;
- mutable Stats::Scalar write_hits;
- mutable Stats::Scalar write_misses;
- mutable Stats::Scalar write_acv;
- mutable Stats::Scalar write_accesses;
- Stats::Formula hits;
- Stats::Formula misses;
- Stats::Formula accesses;
+ struct TlbStats : public Stats::Group
+ {
+ TlbStats(Stats::Group *parent);
+
+ mutable Stats::Scalar read_hits;
+ mutable Stats::Scalar read_misses;
+ mutable Stats::Scalar read_acv;
+ mutable Stats::Scalar read_accesses;
+ mutable Stats::Scalar write_hits;
+ mutable Stats::Scalar write_misses;
+ mutable Stats::Scalar write_acv;
+ mutable Stats::Scalar write_accesses;
+
+ Stats::Formula hits;
+ Stats::Formula misses;
+ Stats::Formula accesses;
+ } stats;
public:
typedef RiscvTLBParams Params;
@@ -98,8 +104,6 @@
void serialize(CheckpointOut &cp) const override;
void unserialize(CheckpointIn &cp) override;
- void regStats() override;
-
Addr translateWithTLB(Addr vaddr, uint16_t asid, Mode mode);
Fault translateAtomic(const RequestPtr &req,
diff --git a/src/arch/x86/tlb.cc b/src/arch/x86/tlb.cc
index 4e9d4be..11ce660 100644
--- a/src/arch/x86/tlb.cc
+++ b/src/arch/x86/tlb.cc
@@ -61,7 +61,7 @@
TLB::TLB(const Params *p)
: BaseTLB(p), configAddress(0), size(p->size),
- tlb(size), lruSeq(0), m5opRange(p->system->m5opRange())
+ tlb(size), lruSeq(0), m5opRange(p->system->m5opRange()), stats(this)
{
if (!size)
fatal("TLBs must have a non-zero size.\n");
@@ -373,18 +373,18 @@
// The vaddr already has the segment base applied.
TlbEntry *entry = lookup(vaddr);
if (mode == Read) {
- rdAccesses++;
+ stats.rdAccesses++;
} else {
- wrAccesses++;
+ stats.wrAccesses++;
}
if (!entry) {
DPRINTF(TLB, "Handling a TLB miss for "
"address %#x at pc %#x.\n",
vaddr, tc->instAddr());
if (mode == Read) {
- rdMisses++;
+ stats.rdMisses++;
} else {
- wrMisses++;
+ stats.wrMisses++;
}
if (FullSystem) {
Fault fault = walker->start(tc, translation, req,
mode);
@@ -518,27 +518,13 @@
return walker;
}
-void
-TLB::regStats()
+TLB::TlbStats::TlbStats(Stats::Group *parent)
+ : Stats::Group(parent),
+ ADD_STAT(rdAccesses, "TLB accesses on read requests"),
+ ADD_STAT(wrAccesses, "TLB accesses on write requests"),
+ ADD_STAT(rdMisses, "TLB misses on read requests"),
+ ADD_STAT(wrMisses, "TLB misses on write requests")
{
- using namespace Stats;
- BaseTLB::regStats();
- rdAccesses
- .name(name() + ".rdAccesses")
- .desc("TLB accesses on read requests");
-
- wrAccesses
- .name(name() + ".wrAccesses")
- .desc("TLB accesses on write requests");
-
- rdMisses
- .name(name() + ".rdMisses")
- .desc("TLB misses on read requests");
-
- wrMisses
- .name(name() + ".wrMisses")
- .desc("TLB misses on write requests");
-
}
void
diff --git a/src/arch/x86/tlb.hh b/src/arch/x86/tlb.hh
index 2a8cabf..955b494 100644
--- a/src/arch/x86/tlb.hh
+++ b/src/arch/x86/tlb.hh
@@ -101,11 +101,15 @@
AddrRange m5opRange;
- // Statistics
- Stats::Scalar rdAccesses;
- Stats::Scalar wrAccesses;
- Stats::Scalar rdMisses;
- Stats::Scalar wrMisses;
+ struct TlbStats : public Stats::Group
+ {
+ TlbStats(Stats::Group *parent);
+
+ Stats::Scalar rdAccesses;
+ Stats::Scalar wrAccesses;
+ Stats::Scalar rdMisses;
+ Stats::Scalar wrMisses;
+ } stats;
Fault translateInt(bool read, RequestPtr req, ThreadContext *tc);
@@ -149,11 +153,6 @@
TlbEntry *insert(Addr vpn, const TlbEntry &entry);
- /*
- * Function to register Stats
- */
- void regStats() override;
-
// Checkpointing
void serialize(CheckpointOut &cp) const override;
void unserialize(CheckpointIn &cp) override;
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/32835
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: Ie2754d861a658fde0acdda30cbcb91e02029e33a
Gerrit-Change-Number: 32835
Gerrit-PatchSet: 1
Gerrit-Owner: Emily Brickey <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
