Anis Peysieux has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/17851

Change subject: mem-cache: Fix RRPV for RRIP
......................................................................

mem-cache: Fix RRPV for RRIP

The RRPV values for RRIP and NRU replacment policies.
Long re-rereference interval was used instead of
distant re-rereference interval and vice-versa.
The btp value permit to choose beetwen distant and
long insertion ratio. A btp value of 0 force the
policy to always insert at a distant re-reference
interval and a btp value of 100 force the policy to
always insert at a long (intermediate) re-rereference
interval.

Change-Id: I516098f73942b769dcc31fe0edfe07c3e9c3effd
---
M src/mem/cache/replacement_policies/ReplacementPolicies.py
M src/mem/cache/replacement_policies/brrip_rp.hh
2 files changed, 6 insertions(+), 2 deletions(-)



diff --git a/src/mem/cache/replacement_policies/ReplacementPolicies.py b/src/mem/cache/replacement_policies/ReplacementPolicies.py
index 0bbf1d1..f40d145 100644
--- a/src/mem/cache/replacement_policies/ReplacementPolicies.py
+++ b/src/mem/cache/replacement_policies/ReplacementPolicies.py
@@ -85,10 +85,10 @@
         "Percentage of blocks to be inserted with long RRPV")

 class RRIPRP(BRRIPRP):
-    btp = 0
+    btp = 100

 class NRURP(BRRIPRP):
-    btp = 0
+    btp = 100
     max_RRPV = 1

 class TreePLRURP(BaseReplacementPolicy):
diff --git a/src/mem/cache/replacement_policies/brrip_rp.hh b/src/mem/cache/replacement_policies/brrip_rp.hh
index 9374742..d374664 100644
--- a/src/mem/cache/replacement_policies/brrip_rp.hh
+++ b/src/mem/cache/replacement_policies/brrip_rp.hh
@@ -66,6 +66,10 @@
     {
         /**
          * Re-Reference Interval Prediction Value.
+         * Some values have specific names (according to the paper):
+         * 0 -> near-immediate re-rereference interval
+         * max_RRPV-1 -> long re-rereference interval
+         * max_RRPV -> distant re-rereference interval
          * A value equal to max_RRPV + 1 indicates an invalid entry.
          */
         int rrpv;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/17851
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: I516098f73942b769dcc31fe0edfe07c3e9c3effd
Gerrit-Change-Number: 17851
Gerrit-PatchSet: 1
Gerrit-Owner: Anis Peysieux <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to