mem: clean up bus/cache DPRINTFs a bit
Not so much noise on failed sends, and more complete info when grepping a trace using an address. --HG-- extra : convert_revision : 05a8261c9452072ca08b906200c6322b33e2b9f1
This commit is contained in:
parent
4a4aa59632
commit
76c4c5fabc
2 changed files with 11 additions and 11 deletions
|
@ -170,8 +170,6 @@ bool
|
|||
Bus::recvTiming(PacketPtr pkt)
|
||||
{
|
||||
short src = pkt->getSrc();
|
||||
DPRINTF(Bus, "recvTiming: packet src %d dest %d addr 0x%x cmd %s\n",
|
||||
src, pkt->getDest(), pkt->getAddr(), pkt->cmdString());
|
||||
|
||||
BusPort *src_port;
|
||||
if (src == defaultId)
|
||||
|
@ -191,10 +189,14 @@ Bus::recvTiming(PacketPtr pkt)
|
|||
(retryList.size() && (!inRetry || src_port != retryList.front()))))
|
||||
{
|
||||
addToRetryList(src_port);
|
||||
DPRINTF(Bus, "recvTiming: Bus is busy, returning false\n");
|
||||
DPRINTF(Bus, "recvTiming: src %d dst %d %s 0x%x BUSY\n",
|
||||
src, pkt->getDest(), pkt->cmdString(), pkt->getAddr());
|
||||
return false;
|
||||
}
|
||||
|
||||
DPRINTF(Bus, "recvTiming: src %d dst %d %s 0x%x\n",
|
||||
src, pkt->getDest(), pkt->cmdString(), pkt->getAddr());
|
||||
|
||||
if (!pkt->isExpressSnoop()) {
|
||||
occupyBus(pkt);
|
||||
}
|
||||
|
@ -243,7 +245,8 @@ Bus::recvTiming(PacketPtr pkt)
|
|||
// Packet not successfully sent. Leave or put it on the retry list.
|
||||
// illegal to block responses... can lead to deadlock
|
||||
assert(!pkt->isResponse());
|
||||
DPRINTF(Bus, "Adding2 a retry to RETRY list %d\n", src);
|
||||
DPRINTF(Bus, "recvTiming: src %d dst %d %s 0x%x TGT RETRY\n",
|
||||
src, pkt->getDest(), pkt->cmdString(), pkt->getAddr());
|
||||
addToRetryList(src_port);
|
||||
return false;
|
||||
}
|
||||
|
|
11
src/mem/cache/cache_impl.hh
vendored
11
src/mem/cache/cache_impl.hh
vendored
|
@ -994,6 +994,10 @@ Cache<TagStore>::handleSnoop(PacketPtr pkt, BlkType *blk,
|
|||
blk->status &= ~bits_to_clear;
|
||||
}
|
||||
|
||||
DPRINTF(Cache, "snooped a %s request for addr %x, %snew state is %i\n",
|
||||
pkt->cmdString(), blockAlign(pkt->getAddr()),
|
||||
respond ? "responding, " : "", invalidate ? 0 : blk->status);
|
||||
|
||||
if (respond) {
|
||||
assert(!pkt->memInhibitAsserted());
|
||||
pkt->assertMemInhibit();
|
||||
|
@ -1013,10 +1017,6 @@ Cache<TagStore>::handleSnoop(PacketPtr pkt, BlkType *blk,
|
|||
if (invalidate) {
|
||||
tags->invalidateBlk(blk);
|
||||
}
|
||||
|
||||
DPRINTF(Cache, "snooped a %s request for addr %x, %snew state is %i\n",
|
||||
pkt->cmdString(), blockAlign(pkt->getAddr()),
|
||||
respond ? "responding, " : "", blk->status);
|
||||
}
|
||||
|
||||
|
||||
|
@ -1384,9 +1384,6 @@ Cache<TagStore>::MemSidePort::sendPacket()
|
|||
MSHR *mshr = dynamic_cast<MSHR*>(pkt->senderState);
|
||||
|
||||
bool success = sendTiming(pkt);
|
||||
DPRINTF(CachePort,
|
||||
"Address %x was %s in sending the timing request\n",
|
||||
pkt->getAddr(), success ? "successful" : "unsuccessful");
|
||||
|
||||
waitingOnRetry = !success;
|
||||
if (waitingOnRetry) {
|
||||
|
|
Loading…
Reference in a new issue