[gem5-dev] changeset in gem5: SE/FS: Get rid of FULL_SYSTEM in mem.

[Gabe Black]

changeset 8be24baf68b8 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=8be24baf68b8
description:
        SE/FS: Get rid of FULL_SYSTEM in mem.

diffstat:

 src/mem/cache/base.cc |  13 +++----------
 src/mem/cache/base.hh |   9 +++------
 2 files changed, 6 insertions(+), 16 deletions(-)

diffs (76 lines):

diff -r ef044a5d7321 -r 8be24baf68b8 src/mem/cache/base.cc
--- a/src/mem/cache/base.cc     Mon Nov 07 01:06:54 2011 -0800
+++ b/src/mem/cache/base.cc     Mon Nov 07 01:13:43 2011 -0800
@@ -38,6 +38,7 @@
 #include "debug/Cache.hh"
 #include "mem/cache/base.hh"
 #include "mem/cache/mshr.hh"
+#include "sim/full_system.hh"
 
 using namespace std;
 
@@ -151,11 +152,7 @@
         const string &cstr = cmd.toString();
 
         hits[access_idx]
-#if FULL_SYSTEM
-            .init(_numCpus + 1)
-#else
-            .init(_numCpus)
-#endif
+            .init(FullSystem ? (_numCpus + 1) : _numCpus)
             .name(name() + "." + cstr + "_hits")
             .desc("number of " + cstr + " hits")
             .flags(total | nozero | nonan)
@@ -192,11 +189,7 @@
         const string &cstr = cmd.toString();
 
         misses[access_idx]
-#if FULL_SYSTEM
-            .init(_numCpus + 1)
-#else
-            .init(_numCpus)
-#endif
+            .init(FullSystem ? (_numCpus + 1) : _numCpus)
             .name(name() + "." + cstr + "_misses")
             .desc("number of " + cstr + " misses")
             .flags(total | nozero | nonan)
diff -r ef044a5d7321 -r 8be24baf68b8 src/mem/cache/base.hh
--- a/src/mem/cache/base.hh     Mon Nov 07 01:06:54 2011 -0800
+++ b/src/mem/cache/base.hh     Mon Nov 07 01:13:43 2011 -0800
@@ -57,6 +57,7 @@
 #include "mem/tport.hh"
 #include "params/BaseCache.hh"
 #include "sim/eventq.hh"
+#include "sim/full_system.hh"
 #include "sim/sim_exit.hh"
 
 class MSHR;
@@ -505,12 +506,10 @@
              * available, meanwhile writeback hit/miss stats are not used
              * in any aggregate hit/miss calculations, so just lump them all
              * in bucket 0 */
-#if FULL_SYSTEM
-        } else if (id == -1) {
+        } else if (FullSystem && id == -1) {
             // Device accesses have id -1
             // lump device accesses into their own bucket
             misses[pkt->cmdToIndex()][_numCpus]++;
-#endif
         } else {
             misses[pkt->cmdToIndex()][id % _numCpus]++;
         }
@@ -533,12 +532,10 @@
         if (pkt->cmd == MemCmd::Writeback) {
             assert(id == -1);
             hits[pkt->cmdToIndex()][0]++;
-#if FULL_SYSTEM
-        } else if (id == -1) {
+        } else if (FullSystem && id == -1) {
             // Device accesses have id -1
             // lump device accesses into their own bucket
             hits[pkt->cmdToIndex()][_numCpus]++;
-#endif
         } else {
             /* the % is necessary in case there are switch cpus */
             hits[pkt->cmdToIndex()][id % _numCpus]++;
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev