Tiago Muck has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/41867 )
Change subject: tests: extend ruby_mem_test
......................................................................
tests: extend ruby_mem_test
Replace ruby_mem_test by these tests which run different
configurations:
ruby_mem_test-garnet: use Garnet
ruby_mem_test-simple: use SimpleNetwork (same as original ruby_mem_test)
ruby_mem_test-simple-extra: use SimpleNetwork with
--simple-physical-channels option
ruby_mem_test-simple-extra-multicore: same as previous using 4 cores
JIRA: https://gem5.atlassian.net/browse/GEM5-920
Change-Id: I7716cd222dd56ddbf06f53f92ec9b568ed5a182c
Signed-off-by: Tiago Mück <[email protected]>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/41867
Reviewed-by: Meatboy 106 <[email protected]>
Maintainer: Bobby Bruce <[email protected]>
Tested-by: kokoro <[email protected]>
---
M tests/gem5/memory/test.py
1 file changed, 49 insertions(+), 8 deletions(-)
Approvals:
Meatboy 106: Looks good to me, approved
Bobby Bruce: Looks good to me, approved
kokoro: Regressions pass
diff --git a/tests/gem5/memory/test.py b/tests/gem5/memory/test.py
index db20ab5..01bd68b 100644
--- a/tests/gem5/memory/test.py
+++ b/tests/gem5/memory/test.py
@@ -68,17 +68,30 @@
)
null_tests = [
- ('garnet_synth_traffic', ['--sim-cycles', '5000000']),
- ('memcheck', ['--maxtick', '2000000000', '--prefetchers']),
- ('ruby_mem_test', ['--abs-max-tick', '20000000',
- '--functional', '10']),
- ('ruby_random_test', ['--maxloads', '5000']),
- ('ruby_direct_test', ['--requests', '50000']),
+ ('garnet_synth_traffic', None, ['--sim-cycles', '5000000']),
+ ('memcheck', None, ['--maxtick', '2000000000', '--prefetchers']),
+ ('ruby_mem_test-garnet', 'ruby_mem_test',
+ ['--abs-max-tick', '20000000', '--functional', '10', \
+ '--network=garnet']),
+ ('ruby_mem_test-simple', 'ruby_mem_test',
+ ['--abs-max-tick', '20000000', '--functional', '10', \
+ '--network=simple']),
+ ('ruby_mem_test-simple-extra', 'ruby_mem_test',
+ ['--abs-max-tick', '20000000', '--functional', '10', \
+ '--network=simple', '--simple-physical-channels']),
+ ('ruby_mem_test-simple-extra-multicore', 'ruby_mem_test',
+ ['--abs-max-tick', '20000000', '--functional', '10', \
+ '--network=simple', '--simple-physical-channels',
+ '--num-cpus=4']),
+ ('ruby_random_test', None, ['--maxloads', '5000']),
+ ('ruby_direct_test', None, ['--requests', '50000']),
]
-for basename_noext, args in null_tests:
+for test_name, basename_noext, args in null_tests:
+ if basename_noext == None:
+ basename_noext = test_name
gem5_verify_config(
- name=basename_noext,
+ name=test_name,
fixtures=(),
verifiers=(),
config=joinpath(config.base_dir, 'configs',
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/41867
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: I7716cd222dd56ddbf06f53f92ec9b568ed5a182c
Gerrit-Change-Number: 41867
Gerrit-PatchSet: 4
Gerrit-Owner: Tiago Muck <[email protected]>
Gerrit-Reviewer: Bobby Bruce <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Meatboy 106 <[email protected]>
Gerrit-Reviewer: Tiago Muck <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
