diff options
author | Daniel R. Carvalho <odanrc@yahoo.com.br> | 2020-01-18 16:19:53 +0100 |
---|---|---|
committer | Daniel Carvalho <odanrc@yahoo.com.br> | 2020-01-21 08:07:05 +0000 |
commit | 519808c02f2d5d780bf09589d2e29803a5af1ade (patch) | |
tree | fd72cd991a2fbedf883fde2ceefc2f8bfc0fe009 /src/mem/cache/prefetch/associative_set_impl.hh | |
parent | 7ac4607385f8490b2a12acaef73f2f31583160f0 (diff) | |
download | gem5-519808c02f2d5d780bf09589d2e29803a5af1ade.tar.xz |
mem-cache: Fix invalidation of prefetchers
Add an invalidation function to the AssociativeSet, so that entries
can be properly invalidated by also invalidating their replacement
data.
Both setInvalid and reset have been merged into invalidate to
indicate users that they are using an incorrect approach by
generating compilation errors, and to match CacheBlk's naming
convention.
Change-Id: I568076a3b5adda8b1311d9498b086c0dab457a14
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/24529
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
Diffstat (limited to 'src/mem/cache/prefetch/associative_set_impl.hh')
-rw-r--r-- | src/mem/cache/prefetch/associative_set_impl.hh | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/mem/cache/prefetch/associative_set_impl.hh b/src/mem/cache/prefetch/associative_set_impl.hh index cc547e2dc..8acae6269 100644 --- a/src/mem/cache/prefetch/associative_set_impl.hh +++ b/src/mem/cache/prefetch/associative_set_impl.hh @@ -87,7 +87,7 @@ AssociativeSet<Entry>::findVictim(Addr addr) Entry* victim = static_cast<Entry*>(replacementPolicy->getVictim( selected_entries)); // There is only one eviction for this replacement - victim->reset(); + invalidate(victim); return victim; } @@ -117,4 +117,12 @@ AssociativeSet<Entry>::insertEntry(Addr addr, bool is_secure, Entry* entry) replacementPolicy->reset(entry->replacementData); } +template<class Entry> +void +AssociativeSet<Entry>::invalidate(Entry* entry) +{ + entry->invalidate(); + replacementPolicy->invalidate(entry->replacementData); +} + #endif//__CACHE_PREFETCH_ASSOCIATIVE_SET_IMPL_HH__ |