diff options
| author | Daniel R. Carvalho <odanrc@yahoo.com.br> | 2018-06-02 15:10:42 +0200 |
|---|---|---|
| committer | Daniel Carvalho <odanrc@yahoo.com.br> | 2018-06-08 09:37:23 +0000 |
| commit | 59505f7305cc3f3b7637233fd2d231bd7f561e80 (patch) | |
| tree | e21dd307531b5f29fd982ccbaa1a2b5b8c2df058 | |
| parent | fa44e21f2eb0ea1be035b2a14bd6ad6ada14375b (diff) | |
| download | gem5-59505f7305cc3f3b7637233fd2d231bd7f561e80.tar.xz | |
mem-cache: Change Cache block tag check
Change tag to address check for compatibility with sector design.
Cache should not use tag, as sector sub-blocks share them, and
it could lead to wrong accesses.
Change-Id: Id1fa26f417595f475c5b5c07ae1f02f5fa0684ba
Reviewed-on: https://gem5-review.googlesource.com/10723
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
| -rw-r--r-- | src/mem/cache/base.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mem/cache/base.cc b/src/mem/cache/base.cc index fdfe37ef4..a8f29e370 100644 --- a/src/mem/cache/base.cc +++ b/src/mem/cache/base.cc @@ -1144,7 +1144,7 @@ BaseCache::handleFill(PacketPtr pkt, CacheBlk *blk, PacketList &writebacks, assert(!blk->isValid()); } else { // existing block... probably an upgrade - assert(blk->tag == tags->extractTag(addr)); + assert(regenerateBlkAddr(blk) == addr); assert(blk->isSecure() == is_secure); // either we're getting new data or the block should already be valid assert(pkt->hasData() || blk->isValid()); |