mem: Remove the cache builder
authorAndreas Hansson <andreas.hansson@arm.com>
Thu, 27 Jun 2013 09:49:50 +0000 (05:49 -0400)
committerAndreas Hansson <andreas.hansson@arm.com>
Thu, 27 Jun 2013 09:49:50 +0000 (05:49 -0400)
This patch removes the redundant cache builder class.

src/mem/cache/SConscript
src/mem/cache/base.cc
src/mem/cache/builder.cc [deleted file]
src/mem/cache/cache.cc
src/mem/config/cache.hh [deleted file]

index e7b0dca0c5b73971fdc918ca37639f281bbc585a..8323602d2c8410f6fa532bcf33183ed7dd7cfe3d 100644 (file)
@@ -38,7 +38,6 @@ SimObject('BaseCache.py')
 Source('base.cc')
 Source('cache.cc')
 Source('blk.cc')
-Source('builder.cc')
 Source('mshr.cc')
 Source('mshr_queue.cc')
 
index 85265b61e2b7b9090c44d7cfa5bee1eaa97ba918..ba981b606b6a82da858d613843cafe9742d540ff 100644 (file)
 #include "cpu/smt.hh"
 #include "debug/Cache.hh"
 #include "debug/Drain.hh"
+#include "mem/cache/tags/fa_lru.hh"
+#include "mem/cache/tags/lru.hh"
 #include "mem/cache/base.hh"
+#include "mem/cache/cache.hh"
 #include "mem/cache/mshr.hh"
 #include "sim/full_system.hh"
 
@@ -766,3 +769,17 @@ BaseCache::drain(DrainManager *dm)
     setDrainState(Drainable::Drained);
     return 0;
 }
+
+BaseCache *
+BaseCacheParams::create()
+{
+    int numSets = size / (assoc * block_size);
+
+    if (numSets == 1) {
+        FALRU *tags = new FALRU(block_size, size, hit_latency);
+        return new Cache<FALRU>(this, tags);
+    } else {
+        LRU *tags = new LRU(numSets, block_size, assoc, hit_latency);
+        return new Cache<LRU>(this, tags);
+    }
+}
diff --git a/src/mem/cache/builder.cc b/src/mem/cache/builder.cc
deleted file mode 100644 (file)
index 7de8fbf..0000000
+++ /dev/null
@@ -1,98 +0,0 @@
-/*
- * Copyright (c) 2003-2005 The Regents of The University of Michigan
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met: redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer;
- * redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution;
- * neither the name of the copyright holders nor the names of its
- * contributors may be used to endorse or promote products derived from
- * this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- *
- * Authors: Erik Hallnor
- *          Nathan Binkert
- */
-
-/**
- * @file
- * Simobject instatiation of caches.
- */
-#include <list>
-#include <vector>
-
-#include "config/the_isa.hh"
-#include "mem/cache/base.hh"
-#include "mem/cache/cache.hh"
-#include "mem/config/cache.hh"
-#include "params/BaseCache.hh"
-
-// Tag Templates
-#if defined(USE_CACHE_LRU)
-#include "mem/cache/tags/lru.hh"
-#endif
-
-#if defined(USE_CACHE_FALRU)
-#include "mem/cache/tags/fa_lru.hh"
-#endif
-
-
-using namespace std;
-
-#define BUILD_CACHE(TAGS, tags)                         \
-    do {                                                \
-        Cache<TAGS> *retval =                           \
-            new Cache<TAGS>(this, tags);            \
-        return retval;                                  \
-    } while (0)
-
-#define BUILD_CACHE_PANIC(x) do {                       \
-        panic("%s not compiled into M5", x);            \
-    } while (0)
-
-#if defined(USE_CACHE_FALRU)
-#define BUILD_FALRU_CACHE do {                              \
-        FALRU *tags = new FALRU(block_size, size, hit_latency); \
-        BUILD_CACHE(FALRU, tags);                           \
-    } while (0)
-#else
-#define BUILD_FALRU_CACHE BUILD_CACHE_PANIC("falru cache")
-#endif
-
-#if defined(USE_CACHE_LRU)
-#define BUILD_LRU_CACHE do {                                            \
-        LRU *tags = new LRU(numSets, block_size, assoc, hit_latency);       \
-        BUILD_CACHE(LRU, tags);                                         \
-    } while (0)
-#else
-#define BUILD_LRU_CACHE BUILD_CACHE_PANIC("lru cache")
-#endif
-
-BaseCache *
-BaseCacheParams::create()
-{
-    int numSets = size / (assoc * block_size);
-
-    if (numSets == 1) {
-        BUILD_FALRU_CACHE;
-    } else {
-        BUILD_LRU_CACHE;
-    }
-
-    return NULL;
-}
index 6a0669e06e24164c75e1e1e72c2ace071538a549..d59a0d44b3a053ba6371de3df6e5903c364042ca 100644 (file)
  * Cache template instantiations.
  */
 
-#include "mem/config/cache.hh"
-
-#if defined(USE_CACHE_LRU)
-#include "mem/cache/tags/lru.hh"
-#endif
-
-#if defined(USE_CACHE_FALRU)
 #include "mem/cache/tags/fa_lru.hh"
-#endif
-
+#include "mem/cache/tags/lru.hh"
 #include "mem/cache/cache_impl.hh"
 
 // Template Instantiations
 #ifndef DOXYGEN_SHOULD_SKIP_THIS
 
-
-#if defined(USE_CACHE_FALRU)
 template class Cache<FALRU>;
-#endif
-
-#if defined(USE_CACHE_LRU)
 template class Cache<LRU>;
-#endif
 
 #endif //DOXYGEN_SHOULD_SKIP_THIS
diff --git a/src/mem/config/cache.hh b/src/mem/config/cache.hh
deleted file mode 100644 (file)
index 0b78394..0000000
+++ /dev/null
@@ -1,39 +0,0 @@
-/*
- * Copyright (c) 2004-2005 The Regents of The University of Michigan
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met: redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer;
- * redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution;
- * neither the name of the copyright holders nor the names of its
- * contributors may be used to endorse or promote products derived from
- * this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- *
- * Authors: Nathan Binkert
- */
-
-/**
- * @file
- * Central location to configure which cache types we want to build
- * into the simulator.  In the future, this should probably be
- * autogenerated by some sort of configuration script.
- */
-#define USE_CACHE_LRU 1
-#define USE_CACHE_FALRU 1
-