[gem5-dev] Change in gem5/gem5[develop]: mem-cache: change Cache debug flag to MSHR in MSHR code

[Tom Rollet (Gerrit) via gem5-dev]

Tom Rollet has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/47039 )

Change subject: mem-cache: change Cache debug flag to MSHR in MSHR code
......................................................................

mem-cache: change Cache debug flag to MSHR in MSHR code

Change-Id: I97ba0d5426d51d913649dcabb6d1ca862e4d137a
---
M src/mem/cache/SConscript
M src/mem/cache/mshr.cc
2 files changed, 7 insertions(+), 6 deletions(-)



diff --git a/src/mem/cache/SConscript b/src/mem/cache/SConscript
index f07a918..0150692 100644
--- a/src/mem/cache/SConscript
+++ b/src/mem/cache/SConscript
@@ -46,10 +46,11 @@
 DebugFlag('CacheTags')
 DebugFlag('CacheVerbose')
 DebugFlag('HWPrefetch')
+DebugFlag('MSHR')

 # CacheTags is so outrageously verbose, printing the cache's entire tag
 # array on each timing access, that you should probably have to ask for
 # it explicitly even above and beyond CacheAll.
 CompoundFlag('CacheAll', ['Cache', 'CacheComp', 'CachePort', 'CacheRepl',
-                          'CacheVerbose', 'HWPrefetch'])
+                          'CacheVerbose', 'HWPrefetch', 'MSHR'])

diff --git a/src/mem/cache/mshr.cc b/src/mem/cache/mshr.cc
index 9dae03d..bb4e8df 100644
--- a/src/mem/cache/mshr.cc
+++ b/src/mem/cache/mshr.cc
@@ -52,7 +52,7 @@
 #include "base/logging.hh"
 #include "base/trace.hh"
 #include "base/types.hh"
-#include "debug/Cache.hh"
+#include "debug/MSHR.hh"
 #include "mem/cache/base.hh"
 #include "mem/request.hh"
 #include "sim/core.hh"
@@ -184,13 +184,13 @@

     if (pkt->cmd == MemCmd::UpgradeReq) {
         pkt->cmd = MemCmd::ReadExReq;
-        DPRINTF(Cache, "Replacing UpgradeReq with ReadExReq\n");
+        DPRINTF(MSHR, "Replacing UpgradeReq with ReadExReq\n");
     } else if (pkt->cmd == MemCmd::SCUpgradeReq) {
         pkt->cmd = MemCmd::SCUpgradeFailReq;
-        DPRINTF(Cache, "Replacing SCUpgradeReq with SCUpgradeFailReq\n");
+        DPRINTF(MSHR, "Replacing SCUpgradeReq with SCUpgradeFailReq\n");
     } else if (pkt->cmd == MemCmd::StoreCondReq) {
         pkt->cmd = MemCmd::StoreCondFailReq;
-        DPRINTF(Cache, "Replacing StoreCondReq with StoreCondFailReq\n");
+        DPRINTF(MSHR, "Replacing StoreCondReq with StoreCondFailReq\n");
     }

     if (!has_data) {
@@ -411,7 +411,7 @@
 bool
 MSHR::handleSnoop(PacketPtr pkt, Counter _order)
 {
-    DPRINTF(Cache, "%s for %s\n", __func__, pkt->print());
+    DPRINTF(MSHR, "%s for %s\n", __func__, pkt->print());

     // when we snoop packets the needsWritable and isInvalidate flags
     // should always be the same, however, this assumes that we never

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/47039
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I97ba0d5426d51d913649dcabb6d1ca862e4d137a
Gerrit-Change-Number: 47039
Gerrit-PatchSet: 1
Gerrit-Owner: Tom Rollet <tom.rol...@huawei.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s