diff options
author | Andreas Hansson <andreas.hansson@arm.com> | 2016-02-24 04:16:57 -0500 |
---|---|---|
committer | Andreas Hansson <andreas.hansson@arm.com> | 2016-02-24 04:16:57 -0500 |
commit | 7958f34797857fecf803fa4a2fcd018a8fffb640 (patch) | |
tree | dbf93b72ae03dd49a915c81388d522b8cb8d75d0 | |
parent | 496a8c6c924b2b970eafa941b2617aca789ebe3c (diff) | |
download | gem5-7958f34797857fecf803fa4a2fcd018a8fffb640.tar.xz |
mem: Ensure that InvalidateReq is not forwarded as ReadExReq
This patch fixes an issue where an InvalidationReq only traversed one
level of the cache hierarchy, and was subsequently turned into a
ReadExReq due to it needing writable, and the command not being
checked for explicitly.
-rw-r--r-- | src/mem/cache/cache.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/mem/cache/cache.cc b/src/mem/cache/cache.cc index e9b909646..724ccc7d6 100644 --- a/src/mem/cache/cache.cc +++ b/src/mem/cache/cache.cc @@ -966,7 +966,8 @@ Cache::getBusPacket(PacketPtr cpu_pkt, CacheBlk *blk, // where the determination the StoreCond fails is delayed due to // all caches not being on the same local bus. cmd = MemCmd::SCUpgradeFailReq; - } else if (cpu_pkt->cmd == MemCmd::WriteLineReq) { + } else if (cpu_pkt->cmd == MemCmd::WriteLineReq || + cpu_pkt->cmd == MemCmd::InvalidateReq) { // forward as invalidate to all other caches, this gives us // the line in Exclusive state, and invalidates all other // copies |