Daniel Carvalho has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/14875
Change subject: mem-cache: Remove old todo about latency in hit function
......................................................................
mem-cache: Remove old todo about latency in hit function
The header and payload delay have already been accounted and
zeroed previous to calling this function. The probe is not
allowed to modify the packet, therefore no extra delays are
added, and it is safe to remove the todo note.
Change-Id: I8ddf7e189fbe609cdec34364f3c013427930daf7
Signed-off-by: Daniel R. Carvalho <[email protected]>
---
M src/mem/cache/base.cc
1 file changed, 0 insertions(+), 2 deletions(-)
diff --git a/src/mem/cache/base.cc b/src/mem/cache/base.cc
index bad24f7..15e5c76 100644
--- a/src/mem/cache/base.cc
+++ b/src/mem/cache/base.cc
@@ -220,8 +220,6 @@
{
if (pkt->needsResponse()) {
pkt->makeTimingResponse();
- // @todo: Make someone pay for this
- pkt->headerDelay = pkt->payloadDelay = 0;
// In this case we are considering request_time that takes
// into account the delay of the xbar, if any, and just
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/14875
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: I8ddf7e189fbe609cdec34364f3c013427930daf7
Gerrit-Change-Number: 14875
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
