Daniel Carvalho has uploaded this change for review. (
https://gem5-review.googlesource.com/10723
Change subject: mem-cache: Change Cache block tag check
......................................................................
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
---
M src/mem/cache/base.cc
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mem/cache/base.cc b/src/mem/cache/base.cc
index fdfe37e..a8f29e3 100644
--- a/src/mem/cache/base.cc
+++ b/src/mem/cache/base.cc
@@ -1144,7 +1144,7 @@
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());
--
To view, visit https://gem5-review.googlesource.com/10723
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Id1fa26f417595f475c5b5c07ae1f02f5fa0684ba
Gerrit-Change-Number: 10723
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Carvalho <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
