mem-cache: Fix increasing replacement count
authorDaniel R. Carvalho <odanrc@yahoo.com.br>
Thu, 18 Apr 2019 12:54:18 +0000 (14:54 +0200)
committerDaniel Carvalho <odanrc@yahoo.com.br>
Fri, 19 Apr 2019 21:39:01 +0000 (21:39 +0000)
Replacements should be increased when there is any evicted
block, which does not necessarily have to be the victim.

For example, assume a superblock contains 4 blocks, and both
A and C are stored compressed (belonging to SB_1). Then F,
from SB2 needs to make room by replacing SB1. If F map to
location 2, the number of replacements should be increased,
even though 2 had no valid blocks:

 Tag      Data          Tag     Data
|SB_1|--|A|X|C|X|  --> |SB_2| |X|F|X|X|
         1 2 3 4               1 2 3 4

Change-Id: I7b3735d28a35faa8d8fa613a1555bb258da65859
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/18208
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>

src/mem/cache/base.cc

index 70e7ba1869125f717b956884db817299f905ef5c..849cff59896761020ef8ae5b7664ebfc038d4596 100644 (file)
@@ -1326,12 +1326,14 @@ BaseCache::allocateBlock(const PacketPtr pkt, PacketList &writebacks)
 
     // The victim will be replaced by a new entry, so increase the replacement
     // counter if a valid block is being replaced
-    if (victim->isValid()) {
-        DPRINTF(Cache, "replacement: replacing %#llx (%s) with %#llx "
-                "(%s): %s\n", regenerateBlkAddr(victim),
-                victim->isSecure() ? "s" : "ns",
-                addr, is_secure ? "s" : "ns",
-                victim->isDirty() ? "writeback" : "clean");
+    if (evict_blks.size() > 0) {
+        for (const auto& blk : evict_blks) {
+            if (blk->isValid()) {
+                DPRINTF(CacheRepl, "Evicting %s (%#llx) to make room for " \
+                        "%#llx (%s)\n", blk->print(), regenerateBlkAddr(blk),
+                        addr, is_secure);
+            }
+        }
 
         replacements++;
     }