From: Kyle Roarty Date: Thu, 13 Aug 2020 22:09:45 +0000 (-0500) Subject: configs: Add parameter for GPU scalar cache mandatory queue size X-Git-Tag: v20.1.0.0~197 X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e111ccaf7000d2aa567c85041cb1cadfe7572d74;p=gem5.git 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 --- 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)