diff options
author | Daniel R. Carvalho <odanrc@yahoo.com.br> | 2018-10-18 15:31:51 +0200 |
---|---|---|
committer | Daniel Carvalho <odanrc@yahoo.com.br> | 2018-11-14 21:02:08 +0000 |
commit | c6e0d8f54f1ce90933f95a7a3a875fed53b8ee3e (patch) | |
tree | 60bb086bf5b96e5c694eb662f0e34abeae26e6ad /src/mem/cache/tags/Tags.py | |
parent | ea0f654923d13a2f81fe7c22eeca4c8cffae947a (diff) | |
download | gem5-c6e0d8f54f1ce90933f95a7a3a875fed53b8ee3e.tar.xz |
mem-cache: Move access latency calculation to Cache
Access latency was not being calculated properly, as it was
always assuming that for hits reads take as long as writes,
and that parallel accesses would produce the same latency
for read and write misses.
By moving the calculation to the Cache we can use the write/
read information, reduce latency variables duplication and
remove Cache dependency from Tags.
The tag lookup latency is still calculated by the Tags.
Change-Id: I71bc68fb5c3515b372c3bf002d61b6f048a45540
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/13697
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Diffstat (limited to 'src/mem/cache/tags/Tags.py')
-rw-r--r-- | src/mem/cache/tags/Tags.py | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/mem/cache/tags/Tags.py b/src/mem/cache/tags/Tags.py index 8e302898c..b34779eef 100644 --- a/src/mem/cache/tags/Tags.py +++ b/src/mem/cache/tags/Tags.py @@ -54,10 +54,6 @@ class BaseTags(ClockedObject): tag_latency = Param.Cycles(Parent.tag_latency, "The tag lookup latency for this cache") - # Get the RAM access latency from the parent (cache) - data_latency = Param.Cycles(Parent.data_latency, - "The data access latency for this cache") - # Get the warmup percentage from the parent (cache) warmup_percentage = Param.Percent(Parent.warmup_percentage, "Percentage of tags to be touched to warm up the cache") |