gem5-dev@gem5.org

The gem5 Developer List

View all threads

[S] Change in gem5/gem5[develop]: configs: fix CHI config for Garnet

TM
Tiago Muck (Gerrit)
Thu, Sep 22, 2022 7:07 PM

Tiago Muck has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/63811?usp=email )

Change subject: configs: fix CHI config for Garnet
......................................................................

configs: fix CHI config for Garnet

CHI.py was not working with Garnet as ruby_system.network.buffer_size
only exists in SimpleNetwork.

Change-Id: I00e952f8881d385e3591c149e08c9c7a53fda2cd
Signed-off-by: Tiago Mück tiago.muck@arm.com

M configs/ruby/CHI.py
1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/configs/ruby/CHI.py b/configs/ruby/CHI.py
index d31d233..df97b92 100644
--- a/configs/ruby/CHI.py
+++ b/configs/ruby/CHI.py
@@ -266,7 +266,8 @@

  ruby_system.network.control_msg_size = params.cntrl_msg_size
  ruby_system.network.data_msg_size = params.data_width
  • ruby_system.network.buffer_size = params.router_buffer_size
  • if options.network == "simple":
  •    ruby_system.network.buffer_size = params.router_buffer_size
    
    # Incorporate the params into options so it's propagated to
    # makeTopology and create_topology the parent scripts
    

--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/63811?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I00e952f8881d385e3591c149e08c9c7a53fda2cd
Gerrit-Change-Number: 63811
Gerrit-PatchSet: 1
Gerrit-Owner: Tiago Muck tiago.muck@arm.com
Gerrit-MessageType: newchange

Tiago Muck has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/63811?usp=email ) Change subject: configs: fix CHI config for Garnet ...................................................................... configs: fix CHI config for Garnet CHI.py was not working with Garnet as ruby_system.network.buffer_size only exists in SimpleNetwork. Change-Id: I00e952f8881d385e3591c149e08c9c7a53fda2cd Signed-off-by: Tiago Mück <tiago.muck@arm.com> --- M configs/ruby/CHI.py 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/configs/ruby/CHI.py b/configs/ruby/CHI.py index d31d233..df97b92 100644 --- a/configs/ruby/CHI.py +++ b/configs/ruby/CHI.py @@ -266,7 +266,8 @@ ruby_system.network.control_msg_size = params.cntrl_msg_size ruby_system.network.data_msg_size = params.data_width - ruby_system.network.buffer_size = params.router_buffer_size + if options.network == "simple": + ruby_system.network.buffer_size = params.router_buffer_size # Incorporate the params into options so it's propagated to # makeTopology and create_topology the parent scripts -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/63811?usp=email To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings Gerrit-Project: public/gem5 Gerrit-Branch: develop Gerrit-Change-Id: I00e952f8881d385e3591c149e08c9c7a53fda2cd Gerrit-Change-Number: 63811 Gerrit-PatchSet: 1 Gerrit-Owner: Tiago Muck <tiago.muck@arm.com> Gerrit-MessageType: newchange