mem: Tidy up the cache debug messages
authorAndreas Hansson <andreas.hansson@arm.com>
Mon, 2 Mar 2015 09:00:37 +0000 (04:00 -0500)
committerAndreas Hansson <andreas.hansson@arm.com>
Mon, 2 Mar 2015 09:00:37 +0000 (04:00 -0500)
Avoid redundant inclusion of the name in the DPRINTF string.

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

index cf55b85912b7d65f5704ee7625b6d2aff4e4057f..b474aeedcf950dc943cd8d2a57eba5e6c187fa72 100644 (file)
@@ -92,13 +92,13 @@ void
 BaseCache::CacheSlavePort::setBlocked()
 {
     assert(!blocked);
-    DPRINTF(CachePort, "Cache port %s blocking new requests\n", name());
+    DPRINTF(CachePort, "Port is blocking new requests\n");
     blocked = true;
     // if we already scheduled a retry in this cycle, but it has not yet
     // happened, cancel it
     if (sendRetryEvent.scheduled()) {
         owner.deschedule(sendRetryEvent);
-        DPRINTF(CachePort, "Cache port %s deschedule retry\n", name());
+        DPRINTF(CachePort, "Port descheduled retry\n");
         mustSendRetry = true;
     }
 }
@@ -107,10 +107,10 @@ void
 BaseCache::CacheSlavePort::clearBlocked()
 {
     assert(blocked);
-    DPRINTF(CachePort, "Cache port %s accepting new requests\n", name());
+    DPRINTF(CachePort, "Port is accepting new requests\n");
     blocked = false;
     if (mustSendRetry) {
-        // @TODO: need to find a better time (next bus cycle?)
+        // @TODO: need to find a better time (next cycle?)
         owner.schedule(sendRetryEvent, curTick() + 1);
     }
 }
@@ -118,7 +118,7 @@ BaseCache::CacheSlavePort::clearBlocked()
 void
 BaseCache::CacheSlavePort::processSendRetry()
 {
-    DPRINTF(CachePort, "Cache port %s sending retry\n", name());
+    DPRINTF(CachePort, "Port is sending retry\n");
 
     // reset the flag and call retry
     mustSendRetry = false;
index bda3df34a9e8825f4eb8b2479d81588cb821b6ee..cd2f552465ed4a3b9a457f44674e6fb5e807c623 100644 (file)
@@ -129,7 +129,8 @@ class BaseCache : public MemObject
          */
         void requestBus(RequestCause cause, Tick time)
         {
-            DPRINTF(CachePort, "Asserting bus request for cause %d\n", cause);
+            DPRINTF(CachePort, "Scheduling request at %llu due to %d\n",
+                    time, cause);
             reqQueue.schedSendEvent(time);
         }
 
index 803b3bad8e3263d76d597e989e960274fdc1c5b7..32eae66d9d5a721e1e0fb5240537a713cac31578 100644 (file)
@@ -261,8 +261,7 @@ Cache<TagStore>::markInService(MSHR *mshr, bool pending_dirty_resp)
     markInServiceInternal(mshr, pending_dirty_resp);
 #if 0
         if (mshr->originalCmd == MemCmd::HardPFReq) {
-            DPRINTF(HWPrefetch, "%s:Marking a HW_PF in service\n",
-                    name());
+            DPRINTF(HWPrefetch, "Marking a HW_PF in service\n");
             //Also clear pending if need be
             if (!prefetcher->havePending())
             {
@@ -324,10 +323,10 @@ Cache<TagStore>::access(PacketPtr pkt, BlkType *&blk,
     // that can modify its value.
     blk = tags->accessBlock(pkt->getAddr(), pkt->isSecure(), lat, id);
 
-    DPRINTF(Cache, "%s%s %x (%s) %s %s\n", pkt->cmdString(),
+    DPRINTF(Cache, "%s%s %x (%s) %s\n", pkt->cmdString(),
             pkt->req->isInstFetch() ? " (ifetch)" : "",
             pkt->getAddr(), pkt->isSecure() ? "s" : "ns",
-            blk ? "hit" : "miss", blk ? blk->print() : "");
+            blk ? "hit " + blk->print() : "miss");
 
     // Writeback handling is special case.  We can write the block into
     // the cache without having a writeable copy (or any copy at all).