cache: remove drainManager because it's not used
authorAnthony Gutierrez <atgutier@umich.edu>
Tue, 29 Jan 2013 01:19:42 +0000 (20:19 -0500)
committerAnthony Gutierrez <atgutier@umich.edu>
Tue, 29 Jan 2013 01:19:42 +0000 (20:19 -0500)
the cache drainManager is set but never cleared, this is because
the cache itself does not need to be drained and thus never
triggers a signalDrainDone(). because the drainManager variable
is not used properly and does not appear to be necessary it has
been removed with this patch.

src/mem/cache/base.cc
src/mem/cache/base.hh

index d126906d0f62464123d2b3b6d4ce544792c71864..476c086ed07831f6191172697f5192dd81d17695 100644 (file)
@@ -77,7 +77,6 @@ BaseCache::BaseCache(const Params *p)
       blocked(0),
       noTargetMSHR(NULL),
       missCount(p->max_miss_count),
-      drainManager(NULL),
       addrRanges(p->addr_ranges.begin(), p->addr_ranges.end()),
       system(p->system)
 {
@@ -752,8 +751,6 @@ BaseCache::drain(DrainManager *dm)
 
     // Set status
     if (count != 0) {
-        drainManager = dm;
-
         setDrainState(Drainable::Draining);
         DPRINTF(Drain, "Cache not drained\n");
         return count;
index e8adacfa12a1bff3a8a9dd21b109c9c3280cb462..b9b42da78942d08711ebe50605092318af45b9f2 100644 (file)
@@ -287,9 +287,6 @@ class BaseCache : public MemObject
     /** The number of misses to trigger an exit event. */
     Counter missCount;
 
-    /** The drain event. */
-    DrainManager *drainManager;
-
     /**
      * The address range to which the cache responds on the CPU side.
      * Normally this is all possible memory addresses. */