From: Daniel R. Carvalho <odanrc@yahoo.com.br>
Date: Sat, 27 Jun 2020 18:51:03 +0000 (+0200)
Subject: mem-cache: Explicitly define threshold of BDI's sub-compressors
X-Git-Tag: v20.1.0.0~180
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=8626fe101ded3e08d0f97c5e1ce74f3165783f76;p=gem5.git

mem-cache: Explicitly define threshold of BDI's sub-compressors

Allow all sub-compressors of BDI to be successful as long as
they are able to compress. Then, BDI's actual size threshold
acts as the cutting point.

This situation arises on any multi compressor; yet, generalizing
this assumption might be too bold.

Change-Id: Iec5057d16d4a7ba5fb573133a30ea10869bd67e0
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/33386
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
---

diff --git a/src/mem/cache/compressors/Compressors.py b/src/mem/cache/compressors/Compressors.py
index 9fa06f210..2497e82e0 100644
--- a/src/mem/cache/compressors/Compressors.py
+++ b/src/mem/cache/compressors/Compressors.py
@@ -148,6 +148,13 @@ class ZeroCompressor(BaseDictionaryCompressor):
 
 class BDI(MultiCompressor):
     encoding_in_tags=True
-    compressors = [ZeroCompressor(), RepeatedQwordsCompressor(),
-        Base64Delta8(), Base64Delta16(), Base64Delta32(), Base32Delta8(),
-        Base32Delta16(), Base16Delta8()]
+    compressors = [
+        ZeroCompressor(size_threshold_percentage=99),
+        RepeatedQwordsCompressor(size_threshold_percentage=99),
+        Base64Delta8(size_threshold_percentage=99),
+        Base64Delta16(size_threshold_percentage=99),
+        Base64Delta32(size_threshold_percentage=99),
+        Base32Delta8(size_threshold_percentage=99),
+        Base32Delta16(size_threshold_percentage=99),
+        Base16Delta8(size_threshold_percentage=99),
+    ]