commit ba3aa067a31f4802e43b024d3c07b32b4dde8fd8
author Tiago Mück <tiago.muck@arm.com> Tue Apr 13 20:16:34 2021 -0500
committer Tiago Muck <tiago.muck@arm.com> Wed Sep 28 18:56:04 2022 +0000
tree 4ff276d71c208583925618459ab78452273ba7a9
parent eb5eab360f616010204a9c1238dd63d5ce9e2cd9

configs: CHI inc transitions_per_cycle

Previous limit may unintentionally throttle performance for controllers
with a large TBE table and high traffic.

Change-Id: I34d6f8727519b259bb3d4a80b1fff6c59197c508
Signed-off-by: Tiago Mück <tiago.muck@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/63672
Maintainer: Bobby Bruce <bbruce@ucdavis.edu>
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>

configs/ruby/CHI_config.py

diff --git a/configs/ruby/CHI_config.py b/configs/ruby/CHI_config.py
index 98f08c3..a95efec 100644
--- a/configs/ruby/CHI_config.py
+++ b/configs/ruby/CHI_config.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2021 ARM Limited
+# Copyright (c) 2021,2022 ARM Limited
 # All rights reserved.
 #
 # The license below extends only to copyright in the software and shall
@@ -206,7 +206,7 @@
         # triggers. To limit the controller performance, tweak other
         # params such as: input port buffer size, cache banks, and output
         # port latency
-        self.transitions_per_cycle = 128
+        self.transitions_per_cycle = 1024
         # This should be set to true in the data cache controller to enable
         # timeouts on unique lines when a store conditional fails
         self.sc_lock_enabled = False
@@ -670,6 +670,7 @@
             responseFromMemory=MessageBuffer(),
             requestToMemory=MessageBuffer(ordered=True),
             reqRdy=TriggerMessageBuffer(),
+            transitions_per_cycle=1024,
         )
 
         self.connectController(self._cntrl)

src/python/gem5/components/cachehierarchies/chi/nodes/abstract_node.py

diff --git a/src/python/gem5/components/cachehierarchies/chi/nodes/abstract_node.py b/src/python/gem5/components/cachehierarchies/chi/nodes/abstract_node.py
index d2bde12..9853174 100644
--- a/src/python/gem5/components/cachehierarchies/chi/nodes/abstract_node.py
+++ b/src/python/gem5/components/cachehierarchies/chi/nodes/abstract_node.py
@@ -77,7 +77,7 @@
         # triggers. To limit the controller performance, tweak other
         # params such as: input port buffer size, cache banks, and output
         # port latency
-        self.transitions_per_cycle = 128
+        self.transitions_per_cycle = 1024
         # This should be set to true in the data cache controller to enable
         # timeouts on unique lines when a store conditional fails
         self.sc_lock_enabled = False