From: Nikos Nikoleris <nikos.nikoleris@arm.com>
Date: Thu, 10 May 2018 10:44:47 +0000 (+0100)
Subject: mem-cache: Move reference count stats update to blk invalidation
X-Git-Tag: v19.0.0.0~2115
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=5f3ed10a03100046b52238e3707df26f1cc1e1dd;p=gem5.git

mem-cache: Move reference count stats update to blk invalidation

The tags in the cache keep track of the number of references to the
blocks as well as the average number of references between an
insertion and the next invalidation. Previously the stats where
updated only on block insertion and invalidations were ignored. This
changes moves the update of the counters to the block invalidation
function.

Change-Id: Ie7672c13813ec278a65232694024d2e5e17c4612
Reviewed-on: https://gem5-review.googlesource.com/10428
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Reviewed-by: Daniel Carvalho <odanrc@yahoo.com.br>
---

diff --git a/src/mem/cache/tags/base.cc b/src/mem/cache/tags/base.cc
index 0087de818..dfd6976fd 100644
--- a/src/mem/cache/tags/base.cc
+++ b/src/mem/cache/tags/base.cc
@@ -78,16 +78,8 @@ BaseTags::insertBlock(PacketPtr pkt, CacheBlk *blk)
     // Get address
     Addr addr = pkt->getAddr();
 
-    // If we're replacing a block that was previously valid update
-    // stats for it. This can't be done in findBlock() because a
-    // found block might not actually be replaced there if the
-    // coherence protocol says it can't be.
     if (blk->isValid()) {
-        totalRefs += blk->refCount;
-        ++sampledRefs;
-
         invalidate(blk);
-        blk->invalidate();
     }
 
     // Previous block, if existed, has been removed, and now we have
diff --git a/src/mem/cache/tags/base.hh b/src/mem/cache/tags/base.hh
index c04329fe9..47bab4323 100644
--- a/src/mem/cache/tags/base.hh
+++ b/src/mem/cache/tags/base.hh
@@ -239,16 +239,21 @@ class BaseTags : public ClockedObject
     }
 
     /**
-     * This function updates the tags when a block is invalidated but
-     * does not invalidate the block itself.
-     * @param blk The block to invalidate.
+     * This function updates the tags when a block is invalidated
+     *
+     * @param blk A valid block to invalidate.
      */
     virtual void invalidate(CacheBlk *blk)
     {
         assert(blk);
         assert(blk->isValid());
+
         tagsInUse--;
         occupancies[blk->srcMasterId]--;
+        totalRefs += blk->refCount;
+        sampledRefs++;
+
+        blk->invalidate();
     }
 
     /**