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>
This commit is contained in:
committed by
Daniel Carvalho
parent
fa44e21f2e
commit
59505f7305
2
src/mem/cache/base.cc
vendored
2
src/mem/cache/base.cc
vendored
@@ -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());
|
||||
|
||||
Reference in New Issue
Block a user