[gem5-dev] [M] Change in gem5/gem5[develop]: tests: Remove long/nightly ARM Boot tests

2022-11-01 Thread Bobby Bruce (Gerrit) via gem5-dev
Bobby Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/65191?usp=email )


Change subject: tests: Remove long/nightly ARM Boot tests
..

tests: Remove long/nightly ARM Boot tests

The long/nightly tests are failing due to timeout (e.g.:
https://jenkins.gem5.org/job/nightly/398/). We must therefore be more
careful about what we test on a nightly basis.

Each of these ARM boot tests takes an hour and, generally, are largely
the same (just with different CPU cores, cache hierarchy, and memory
system). Given this is largely redundant, some of these tests have been
removed to save on testing time.

Change-Id: I8d80d3e0869aca67aa7279a164bdce85d20f3682
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/65191
Maintainer: Jason Lowe-Power 
Reviewed-by: Jason Lowe-Power 
Tested-by: kokoro 
---
M tests/gem5/arm-boot-tests/test_linux_boot.py
1 file changed, 23 insertions(+), 57 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass




diff --git a/tests/gem5/arm-boot-tests/test_linux_boot.py  
b/tests/gem5/arm-boot-tests/test_linux_boot.py

index 9a6f671..6115bb2 100644
--- a/tests/gem5/arm-boot-tests/test_linux_boot.py
+++ b/tests/gem5/arm-boot-tests/test_linux_boot.py
@@ -185,32 +185,8 @@

 test_boot(
 cpu="atomic",
-num_cpus=1,
-mem_system="classic",
-memory_class="SingleChannelDDR3_1600",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=1,
-mem_system="classic",
-memory_class="SingleChannelDDR3_2133",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="o3",
-num_cpus=1,
-mem_system="classic",
-memory_class="DualChannelDDR3_1600",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
 num_cpus=4,
-mem_system="classic",
+mem_system="no_cache",
 memory_class="HBM2Stack",
 length=constants.long_tag,
 )
@@ -218,39 +194,7 @@
 test_boot(
 cpu="timing",
 num_cpus=2,
-mem_system="classic",
-memory_class="DualChannelDDR4_2400",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=2,
-mem_system="no_cache",
-memory_class="DualChannelDDR4_2400",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=2,
 mem_system="chi",
 memory_class="DualChannelDDR4_2400",
 length=constants.long_tag,
 )
-
-test_boot(
-cpu="timing",
-num_cpus=2,
-mem_system="mesi_two_level",
-memory_class="DualChannelDDR4_2400",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=2,
-mem_system="mi_example",
-memory_class="DualChannelDDR4_2400",
-length=constants.long_tag,
-)

--
To view, visit  
https://gem5-review.googlesource.com/c/public/gem5/+/65191?usp=email
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: I8d80d3e0869aca67aa7279a164bdce85d20f3682
Gerrit-Change-Number: 65191
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby Bruce 
Gerrit-Reviewer: Bobby Bruce 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: kokoro 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org


[gem5-dev] [M] Change in gem5/gem5[develop]: tests: Remove long/nightly ARM Boot tests

2022-11-01 Thread Bobby Bruce (Gerrit) via gem5-dev
Bobby Bruce has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/65191?usp=email )



Change subject: tests: Remove long/nightly ARM Boot tests
..

tests: Remove long/nightly ARM Boot tests

The long/nightly tests are failing due to timeout (e.g.:
https://jenkins.gem5.org/job/nightly/398/). We must therefore be more
careful about what we test on a nightly basis.

Each of these ARM boot tests takes an hour and, generally, are largely
the same (just with different CPU cores, cache hierarchy, and memory
system). Given this is largely redundant, some of these tests have been
removed to save on testing time.

Change-Id: I8d80d3e0869aca67aa7279a164bdce85d20f3682
---
M tests/gem5/arm-boot-tests/test_linux_boot.py
1 file changed, 19 insertions(+), 57 deletions(-)



diff --git a/tests/gem5/arm-boot-tests/test_linux_boot.py  
b/tests/gem5/arm-boot-tests/test_linux_boot.py

index 9a6f671..6115bb2 100644
--- a/tests/gem5/arm-boot-tests/test_linux_boot.py
+++ b/tests/gem5/arm-boot-tests/test_linux_boot.py
@@ -185,32 +185,8 @@

 test_boot(
 cpu="atomic",
-num_cpus=1,
-mem_system="classic",
-memory_class="SingleChannelDDR3_1600",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=1,
-mem_system="classic",
-memory_class="SingleChannelDDR3_2133",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="o3",
-num_cpus=1,
-mem_system="classic",
-memory_class="DualChannelDDR3_1600",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
 num_cpus=4,
-mem_system="classic",
+mem_system="no_cache",
 memory_class="HBM2Stack",
 length=constants.long_tag,
 )
@@ -218,39 +194,7 @@
 test_boot(
 cpu="timing",
 num_cpus=2,
-mem_system="classic",
-memory_class="DualChannelDDR4_2400",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=2,
-mem_system="no_cache",
-memory_class="DualChannelDDR4_2400",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=2,
 mem_system="chi",
 memory_class="DualChannelDDR4_2400",
 length=constants.long_tag,
 )
-
-test_boot(
-cpu="timing",
-num_cpus=2,
-mem_system="mesi_two_level",
-memory_class="DualChannelDDR4_2400",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=2,
-mem_system="mi_example",
-memory_class="DualChannelDDR4_2400",
-length=constants.long_tag,
-)

--
To view, visit  
https://gem5-review.googlesource.com/c/public/gem5/+/65191?usp=email
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: I8d80d3e0869aca67aa7279a164bdce85d20f3682
Gerrit-Change-Number: 65191
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby Bruce 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org