changeset 348411ec525a in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=348411ec525a
description:
        sim: Call regStats of base-class as well

        We want to extend the stats of objects hierarchically and thus it is 
necessary
        to register the statistics of the base-class(es), as well.  For now, 
these are
        empty, but generic stats will be added there.

        Patch originally provided by Akash Bagdia at ARM Ltd.

diffstat:

 src/arch/arm/table_walker.cc               |  2 ++
 src/arch/arm/tlb.cc                        |  1 +
 src/cpu/base.cc                            |  2 ++
 src/cpu/testers/traffic_gen/traffic_gen.cc |  2 ++
 src/dev/net/etherdevice.cc                 |  2 ++
 src/dev/pci/copy_engine.cc                 |  2 ++
 src/dev/storage/ide_disk.cc                |  2 ++
 src/mem/abstract_mem.cc                    |  2 ++
 src/mem/cache/base.cc                      |  2 ++
 src/mem/cache/prefetch/base.cc             |  2 ++
 src/mem/cache/tags/base.cc                 |  3 +++
 src/mem/comm_monitor.cc                    |  2 ++
 src/mem/xbar.cc                            |  2 ++
 src/sim/process.cc                         |  2 ++
 src/sim/system.cc                          |  2 ++
 15 files changed, 30 insertions(+), 0 deletions(-)

diffs (181 lines):

diff -r 4ad81380c5e8 -r 348411ec525a src/arch/arm/table_walker.cc
--- a/src/arch/arm/table_walker.cc      Mon Jun 06 00:18:34 2016 -0400
+++ b/src/arch/arm/table_walker.cc      Mon Jun 06 17:16:43 2016 +0100
@@ -2122,6 +2122,8 @@
 void
 TableWalker::regStats()
 {
+    ClockedObject::regStats();
+
     statWalks
         .name(name() + ".walks")
         .desc("Table walker walks requested")
diff -r 4ad81380c5e8 -r 348411ec525a src/arch/arm/tlb.cc
--- a/src/arch/arm/tlb.cc       Mon Jun 06 00:18:34 2016 -0400
+++ b/src/arch/arm/tlb.cc       Mon Jun 06 17:16:43 2016 +0100
@@ -417,6 +417,7 @@
 void
 TLB::regStats()
 {
+    BaseTLB::regStats();
     instHits
         .name(name() + ".inst_hits")
         .desc("ITB inst hits")
diff -r 4ad81380c5e8 -r 348411ec525a src/cpu/base.cc
--- a/src/cpu/base.cc   Mon Jun 06 00:18:34 2016 -0400
+++ b/src/cpu/base.cc   Mon Jun 06 17:16:43 2016 +0100
@@ -397,6 +397,8 @@
 void
 BaseCPU::regStats()
 {
+    MemObject::regStats();
+
     using namespace Stats;
 
     numCycles
diff -r 4ad81380c5e8 -r 348411ec525a src/cpu/testers/traffic_gen/traffic_gen.cc
--- a/src/cpu/testers/traffic_gen/traffic_gen.cc        Mon Jun 06 00:18:34 
2016 -0400
+++ b/src/cpu/testers/traffic_gen/traffic_gen.cc        Mon Jun 06 17:16:43 
2016 +0100
@@ -526,6 +526,8 @@
 void
 TrafficGen::regStats()
 {
+    ClockedObject::regStats();
+
     // Initialise all the stats
     using namespace Stats;
 
diff -r 4ad81380c5e8 -r 348411ec525a src/dev/net/etherdevice.cc
--- a/src/dev/net/etherdevice.cc        Mon Jun 06 00:18:34 2016 -0400
+++ b/src/dev/net/etherdevice.cc        Mon Jun 06 17:16:43 2016 +0100
@@ -36,6 +36,8 @@
 void
 EtherDevice::regStats()
 {
+    PciDevice::regStats();
+
     txBytes
         .name(name() + ".txBytes")
         .desc("Bytes Transmitted")
diff -r 4ad81380c5e8 -r 348411ec525a src/dev/pci/copy_engine.cc
--- a/src/dev/pci/copy_engine.cc        Mon Jun 06 00:18:34 2016 -0400
+++ b/src/dev/pci/copy_engine.cc        Mon Jun 06 17:16:43 2016 +0100
@@ -425,6 +425,8 @@
 void
 CopyEngine::regStats()
 {
+    PciDevice::regStats();
+
     using namespace Stats;
     bytesCopied
         .init(regs.chanCount)
diff -r 4ad81380c5e8 -r 348411ec525a src/dev/storage/ide_disk.cc
--- a/src/dev/storage/ide_disk.cc       Mon Jun 06 00:18:34 2016 -0400
+++ b/src/dev/storage/ide_disk.cc       Mon Jun 06 17:16:43 2016 +0100
@@ -392,6 +392,8 @@
 void
 IdeDisk::regStats()
 {
+    SimObject::regStats();
+
     using namespace Stats;
     dmaReadFullPages
         .name(name() + ".dma_read_full_pages")
diff -r 4ad81380c5e8 -r 348411ec525a src/mem/abstract_mem.cc
--- a/src/mem/abstract_mem.cc   Mon Jun 06 00:18:34 2016 -0400
+++ b/src/mem/abstract_mem.cc   Mon Jun 06 17:16:43 2016 +0100
@@ -79,6 +79,8 @@
 void
 AbstractMemory::regStats()
 {
+    MemObject::regStats();
+
     using namespace Stats;
 
     assert(system());
diff -r 4ad81380c5e8 -r 348411ec525a src/mem/cache/base.cc
--- a/src/mem/cache/base.cc     Mon Jun 06 00:18:34 2016 -0400
+++ b/src/mem/cache/base.cc     Mon Jun 06 17:16:43 2016 +0100
@@ -176,6 +176,8 @@
 void
 BaseCache::regStats()
 {
+    MemObject::regStats();
+
     using namespace Stats;
 
     // Hit statistics
diff -r 4ad81380c5e8 -r 348411ec525a src/mem/cache/prefetch/base.cc
--- a/src/mem/cache/prefetch/base.cc    Mon Jun 06 00:18:34 2016 -0400
+++ b/src/mem/cache/prefetch/base.cc    Mon Jun 06 17:16:43 2016 +0100
@@ -74,6 +74,8 @@
 void
 BasePrefetcher::regStats()
 {
+    ClockedObject::regStats();
+
     pfIssued
         .name(name() + ".num_hwpf_issued")
         .desc("number of hwpf issued")
diff -r 4ad81380c5e8 -r 348411ec525a src/mem/cache/tags/base.cc
--- a/src/mem/cache/tags/base.cc        Mon Jun 06 00:18:34 2016 -0400
+++ b/src/mem/cache/tags/base.cc        Mon Jun 06 17:16:43 2016 +0100
@@ -71,7 +71,10 @@
 void
 BaseTags::regStats()
 {
+    ClockedObject::regStats();
+
     using namespace Stats;
+
     replacements
         .init(maxThreadsPerCPU)
         .name(name() + ".replacements")
diff -r 4ad81380c5e8 -r 348411ec525a src/mem/comm_monitor.cc
--- a/src/mem/comm_monitor.cc   Mon Jun 06 00:18:34 2016 -0400
+++ b/src/mem/comm_monitor.cc   Mon Jun 06 17:16:43 2016 +0100
@@ -384,6 +384,8 @@
 void
 CommMonitor::regStats()
 {
+    MemObject::regStats();
+
     // Initialise all the monitor stats
     using namespace Stats;
 
diff -r 4ad81380c5e8 -r 348411ec525a src/mem/xbar.cc
--- a/src/mem/xbar.cc   Mon Jun 06 00:18:34 2016 -0400
+++ b/src/mem/xbar.cc   Mon Jun 06 17:16:43 2016 +0100
@@ -541,6 +541,8 @@
 void
 BaseXBar::regStats()
 {
+    ClockedObject::regStats();
+
     using namespace Stats;
 
     transDist
diff -r 4ad81380c5e8 -r 348411ec525a src/sim/process.cc
--- a/src/sim/process.cc        Mon Jun 06 00:18:34 2016 -0400
+++ b/src/sim/process.cc        Mon Jun 06 17:16:43 2016 +0100
@@ -191,6 +191,8 @@
 void
 Process::regStats()
 {
+    SimObject::regStats();
+
     using namespace Stats;
 
     num_syscalls
diff -r 4ad81380c5e8 -r 348411ec525a src/sim/system.cc
--- a/src/sim/system.cc Mon Jun 06 00:18:34 2016 -0400
+++ b/src/sim/system.cc Mon Jun 06 17:16:43 2016 +0100
@@ -393,6 +393,8 @@
 void
 System::regStats()
 {
+    MemObject::regStats();
+
     for (uint32_t j = 0; j < numWorkIds ; j++) {
         workItemStats[j] = new Stats::Histogram();
         stringstream namestr;
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to