summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel R. Carvalho <odanrc@yahoo.com.br>2018-10-03 14:55:31 +0200
committerDaniel Carvalho <odanrc@yahoo.com.br>2018-10-05 18:42:49 +0000
commite2c8d06289354b539358c2425c28f2939d3ddf48 (patch)
treef1fd55e938a24ccb5f58e923604d32c575af72cd
parent079a16d16373ff739785a923649180a20f977489 (diff)
downloadgem5-e2c8d06289354b539358c2425c28f2939d3ddf48.tar.xz
mem-cache: Make checking function const in FALRU
The checking function should not be able to modify either the head and tail pointers nor should it modify its class. Change-Id: I2ad495f0c8c6b778d48512143e94b4c9a353f22e Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br> Reviewed-on: https://gem5-review.googlesource.com/c/13209 Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com> Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
-rw-r--r--src/mem/cache/tags/fa_lru.cc4
-rw-r--r--src/mem/cache/tags/fa_lru.hh2
2 files changed, 3 insertions, 3 deletions
diff --git a/src/mem/cache/tags/fa_lru.cc b/src/mem/cache/tags/fa_lru.cc
index 60998e254..604bd7d07 100644
--- a/src/mem/cache/tags/fa_lru.cc
+++ b/src/mem/cache/tags/fa_lru.cc
@@ -283,10 +283,10 @@ FALRUParams::create()
}
void
-FALRU::CacheTracking::check(FALRUBlk *head, FALRUBlk *tail)
+FALRU::CacheTracking::check(const FALRUBlk *head, const FALRUBlk *tail) const
{
#ifdef FALRU_DEBUG
- FALRUBlk* blk = head;
+ const FALRUBlk* blk = head;
unsigned curr_size = 0;
unsigned tracked_cache_size = minTrackedSize;
CachesMask in_caches_mask = inAllCachesMask;
diff --git a/src/mem/cache/tags/fa_lru.hh b/src/mem/cache/tags/fa_lru.hh
index 03af572b8..43e58040f 100644
--- a/src/mem/cache/tags/fa_lru.hh
+++ b/src/mem/cache/tags/fa_lru.hh
@@ -335,7 +335,7 @@ class FALRU : public BaseTags
* @param head the MRU block of the actual cache
* @param head the LRU block of the actual cache
*/
- void check(FALRUBlk *head, FALRUBlk *tail);
+ void check(const FALRUBlk *head, const FALRUBlk *tail) const;
/**
* Register the stats for this object.