Daniel Carvalho has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/14878
Change subject: mem-cache: Make CleanEvict only access tag array
......................................................................
mem-cache: Make CleanEvict only access tag array
Previous latency calculation for CleanEvict included the latency to
access the data array, if the block had been found.
Change-Id: I7365a915500b5d7ab636d49a9acc627072a7f58e
Signed-off-by: Daniel R. Carvalho <[email protected]>
---
M src/mem/cache/base.cc
1 file changed, 4 insertions(+), 0 deletions(-)
diff --git a/src/mem/cache/base.cc b/src/mem/cache/base.cc
index 7198754..b54e8c9 100644
--- a/src/mem/cache/base.cc
+++ b/src/mem/cache/base.cc
@@ -1042,6 +1042,10 @@
std::max(cyclesToTicks(tag_latency),
(uint64_t)pkt->payloadDelay));
return true;
} else if (pkt->cmd == MemCmd::CleanEvict) {
+ // A CleanEvict does not need to access the data array, so fix the
+ // latency to reflect that.
+ lat = ticksToCycles(clockEdge() + pkt->headerDelay) + tag_latency;
+
if (blk) {
// Found the block in the tags, need to stop CleanEvict from
// propagating further down the hierarchy. Returning true will
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/14878
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: I7365a915500b5d7ab636d49a9acc627072a7f58e
Gerrit-Change-Number: 14878
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
