Matt Sinclair has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/57535 )
Change subject: tests,configs,mem-ruby: Handle num DMAs in GPU Ruby tester
......................................................................
tests,configs,mem-ruby: Handle num DMAs in GPU Ruby tester
Currently VIPER does not support partial cache line writes,
so we need numDMAs to be 0 for it to work with the GPU Ruby
tester. To enable this, change the default value to -1 to
allow us to identify when a user wants more DMAs.
Change-Id: I0a31f66c831f0379544c15bd7364f185e1edb1b2
---
M configs/example/ruby_gpu_random_test.py
1 file changed, 36 insertions(+), 4 deletions(-)
diff --git a/configs/example/ruby_gpu_random_test.py
b/configs/example/ruby_gpu_random_test.py
index 0763454..7fcaeeb 100644
--- a/configs/example/ruby_gpu_random_test.py
+++ b/configs/example/ruby_gpu_random_test.py
@@ -79,7 +79,7 @@
help="Random seed number. Default value (i.e., 0)
means \
using runtime-specific value")
parser.add_argument("--log-file", type=str, default="gpu-ruby-test.log")
-parser.add_argument("--num-dmas", type=int, default=0,
+parser.add_argument("--num-dmas", type=int, default=-1,
help="The number of DMA engines to use in tester
config.")
args = parser.parse_args()
@@ -108,7 +108,13 @@
args.wf_size = 1
args.wavefronts_per_cu = 1
args.num_cpus = 1
- args.num_dmas = 1
+ # if user didn't specify number of DMAs, then assume 0
+ if args.num_dmas < 1:
+ # currently VIPER does not support partial cache line writes,
+ # so we need numDMAs to be 0 for it
+ args.num_dmas = 0
+ else:
+ args.num_dmas = 1
args.cu_per_sqc = 1
args.cu_per_scalar_cache = 1
args.num_compute_units = 1
@@ -117,7 +123,13 @@
args.wf_size = 16
args.wavefronts_per_cu = 4
args.num_cpus = 4
- args.num_dmas = 2
+ # if user didn't specify number of DMAs, then assume 0
+ if args.num_dmas < 1:
+ # currently VIPER does not support partial cache line writes,
+ # so we need numDMAs to be 0 for it
+ args.num_dmas = 0
+ else:
+ args.num_dmas = 2
args.cu_per_sqc = 4
args.cu_per_scalar_cache = 4
args.num_compute_units = 4
@@ -126,7 +138,13 @@
args.wf_size = 32
args.wavefronts_per_cu = 4
args.num_cpus = 4
- args.num_dmas = 4
+ # if user didn't specify number of DMAs, then assume 0
+ if args.num_dmas < 1:
+ # currently VIPER does not support partial cache line writes,
+ # so we need numDMAs to be 0 for it
+ args.num_dmas = 0
+ else:
+ args.num_dmas = 4
args.cu_per_sqc = 4
args.cu_per_scalar_cache = 4
args.num_compute_units = 8
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/57535
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: I0a31f66c831f0379544c15bd7364f185e1edb1b2
Gerrit-Change-Number: 57535
Gerrit-PatchSet: 1
Gerrit-Owner: Matt Sinclair <mattdsincl...@gmail.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
