diff options
| author | Daniel R. Carvalho <odanrc@yahoo.com.br> | 2018-12-04 13:28:04 +0100 |
|---|---|---|
| committer | Daniel Carvalho <odanrc@yahoo.com.br> | 2019-03-07 11:13:37 +0000 |
| commit | 6c00d57cce58973094e9cacd1bf3431a3407e866 (patch) | |
| tree | 15507a5e13d3e4cc43e0e37bd583372b1ec37217 /src/mem/cache | |
| parent | 050545fc180fc9bdb27d9c0813f9d0043a89e7f7 (diff) | |
| download | gem5-6c00d57cce58973094e9cacd1bf3431a3407e866.tar.xz | |
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 <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/14875
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Diffstat (limited to 'src/mem/cache')
| -rw-r--r-- | src/mem/cache/base.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/mem/cache/base.cc b/src/mem/cache/base.cc index 5b4307b76..0949e498c 100644 --- a/src/mem/cache/base.cc +++ b/src/mem/cache/base.cc @@ -220,9 +220,11 @@ void BaseCache::handleTimingReqHit(PacketPtr pkt, CacheBlk *blk, Tick request_time) { if (pkt->needsResponse()) { + // These delays should have been consumed by now + assert(pkt->headerDelay == 0); + assert(pkt->payloadDelay == 0); + 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 |