[gem5-dev] Change in gem5/gem5[develop]: tests, learning-gem5: Moved MIPS ISA test from part 1 to long

[Bobby R. Bruce (Gerrit)]

Bobby R. Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/26844 )

Change subject: tests,learning-gem5: Moved MIPS ISA test from part 1 to long
......................................................................
tests,learning-gem5: Moved MIPS ISA test from part 1 to long

The learning gem5 part 1 tests were the only "quick" tests requiring the
MIPS ISA to be compilated. This is a big cost for two very simple tests.
The MIPS ISA tests for learning gem5 part 1 have been moved to the
"long" tests.

Change-Id: I694541b4c7ea84e91262f29c67fb5ec2bbbc6fec
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/26844
Reviewed-by: Jason Lowe-Power <power...@gmail.com>
Maintainer: Bobby R. Bruce <bbr...@ucdavis.edu>
Tested-by: kokoro <noreply+kok...@google.com>
---
M tests/gem5/learning_gem5/part1_test.py
1 file changed, 26 insertions(+), 2 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved
  Bobby R. Bruce: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/tests/gem5/learning_gem5/part1_test.py  
b/tests/gem5/learning_gem5/part1_test.py
index a0ce56c..b57ba17 100644
--- a/tests/gem5/learning_gem5/part1_test.py
+++ b/tests/gem5/learning_gem5/part1_test.py
@@ -28,19 +28,43 @@

 config_path =  
joinpath(config.base_dir, 'configs', 'learning_gem5', 'part1')

+# The "quick" simple tests.
 gem5_verify_config(
     name='simple_test',
     verifiers = (),
     config=joinpath(config_path, 'simple.py'),
     config_args = [],
-    valid_isas=('X86', 'RISCV', 'ARM', 'MIPS'),
+    length = constants.quick_tag,
+    valid_isas=('X86', 'RISCV', 'ARM'),
 )

+# The "long" simple tests.
+gem5_verify_config(
+    name='simple_test',
+    verifiers = (),
+    config=joinpath(config_path, 'simple.py'),
+    config_args = [],
+    length = constants.long_tag,
+    valid_isas=('MIPS',),
+)
+
+# The "quick" two level tests.
 gem5_verify_config(
     name='two_level_test',
     verifiers = (),
     config=joinpath(config_path, 'two_level.py'),
     config_args = [],
-    valid_isas=('X86', 'RISCV', 'ARM', 'MIPS'),
+    length = constants.quick_tag,
+    valid_isas=('X86', 'RISCV', 'ARM'),
+)
+
+# The "long" two level tests.
+gem5_verify_config(
+    name='two_level_test',
+    verifiers = (),
+    config=joinpath(config_path, 'two_level.py'),
+    config_args = [],
+    length = constants.long_tag,
+    valid_isas=('MIPS',),
 )


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/26844
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: I694541b4c7ea84e91262f29c67fb5ec2bbbc6fec
Gerrit-Change-Number: 26844
Gerrit-PatchSet: 3
Gerrit-Owner: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Giacomo Travaglini <giacomo.travagl...@arm.com>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev