Anis Peysieux has submitted this change and it was merged. (
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
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/17851
Reviewed-by: Daniel Carvalho <[email protected]>
Reviewed-by: Nikos Nikoleris <[email protected]>
Maintainer: Jason Lowe-Power <[email protected]>
Maintainer: Nikos Nikoleris <[email protected]>
---
M src/mem/cache/replacement_policies/ReplacementPolicies.py
M src/mem/cache/replacement_policies/brrip_rp.hh
2 files changed, 6 insertions(+), 2 deletions(-)
Approvals:
Nikos Nikoleris: Looks good to me, approved; Looks good to me, approved
Daniel Carvalho: Looks good to me, approved
Jason Lowe-Power: Looks good to me, approved
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: 2
Gerrit-Owner: Anis Peysieux <[email protected]>
Gerrit-Reviewer: Anis Peysieux <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Nikos Nikoleris <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
