From e111ccaf7000d2aa567c85041cb1cadfe7572d74 Mon Sep 17 00:00:00 2001 From: Kyle Roarty Date: Thu, 13 Aug 2020 17:09:45 -0500 Subject: [PATCH] configs: Add parameter for GPU scalar cache mandatory queue size There was a missing option (--buffers-size) used to set the mandatory queue size for the scalar controllers. This patch renames the option to be more clear, and adds it to the argument parser. Default of 128 taken from the implementation on the GCN staging branch Change-Id: I58b6b57be07498cdf6e39c0bb85982674ec4caa6 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/32676 Reviewed-by: Matt Sinclair Maintainer: Anthony Gutierrez Tested-by: kokoro --- configs/ruby/GPU_VIPER.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/configs/ruby/GPU_VIPER.py b/configs/ruby/GPU_VIPER.py index 50ccd2b25..6a6dec53b 100644 --- a/configs/ruby/GPU_VIPER.py +++ b/configs/ruby/GPU_VIPER.py @@ -399,6 +399,9 @@ def define_options(parser): parser.add_option("--noL1", action = "store_true", default = False, help = "bypassL1") + parser.add_option("--scalar-buffer-size", type = 'int', default = 128, + help="Size of the mandatory queue in the GPU scalar " + "cache controller") def create_system(options, full_system, system, dma_devices, bootmem, ruby_system): @@ -676,7 +679,7 @@ def create_system(options, full_system, system, dma_devices, bootmem, scalar_cntrl.responseToSQC.slave = ruby_system.network.master scalar_cntrl.mandatoryQueue = \ - MessageBuffer(buffer_size=options.buffers_size) + MessageBuffer(buffer_size=options.scalar_buffer_size) gpuCluster.add(scalar_cntrl) -- 2.30.2