mem: Add checks to sendTimingReq in cache
A small fix to ensure the return value is not ignored.
This commit is contained in:
9
src/mem/cache/cache_impl.hh
vendored
9
src/mem/cache/cache_impl.hh
vendored
@@ -462,7 +462,9 @@ Cache<TagStore>::recvTimingReq(PacketPtr pkt)
|
||||
|
||||
// Just forward the packet if caches are disabled.
|
||||
if (system->bypassCaches()) {
|
||||
memSidePort->sendTimingReq(pkt);
|
||||
// @todo This should really enqueue the packet rather
|
||||
bool M5_VAR_USED success = memSidePort->sendTimingReq(pkt);
|
||||
assert(success);
|
||||
return true;
|
||||
}
|
||||
|
||||
@@ -483,7 +485,10 @@ Cache<TagStore>::recvTimingReq(PacketPtr pkt)
|
||||
snoopPkt->busFirstWordDelay = snoopPkt->busLastWordDelay = 0;
|
||||
snoopPkt->setExpressSnoop();
|
||||
snoopPkt->assertMemInhibit();
|
||||
memSidePort->sendTimingReq(snoopPkt);
|
||||
bool M5_VAR_USED success = memSidePort->sendTimingReq(snoopPkt);
|
||||
// the packet is marked inhibited and will thus bypass any
|
||||
// flow control
|
||||
assert(success);
|
||||
// main memory will delete snoopPkt
|
||||
}
|
||||
// since we're the official target but we aren't responding,
|
||||
|
||||
Reference in New Issue
Block a user