[gem5-dev] [M] Change in gem5/gem5[develop]: tests: Remove long/nightly X86 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/+/65192?usp=email )


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

tests: Remove long/nightly X86 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 X86 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
remove dto save on testing time.

Change-Id: I761fca1aa5e111a03183f83d4e326aaea1bdbc3a
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/65192
Reviewed-by: Jason Lowe-Power 
Maintainer: Bobby Bruce 
Tested-by: kokoro 
---
M tests/gem5/x86-boot-tests/test_linux_boot.py
1 file changed, 23 insertions(+), 35 deletions(-)

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




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

index ecda2ff..76d593b 100644
--- a/tests/gem5/x86-boot-tests/test_linux_boot.py
+++ b/tests/gem5/x86-boot-tests/test_linux_boot.py
@@ -150,39 +150,13 @@

  The long (Nightly) tests 

-test_boot(
-cpu="atomic",
-num_cpus=1,
-mem_system="classic",
-memory_class="SingleChannelHBM",
-boot_type="init",
-length=constants.long_tag,
-)

 test_boot(
 cpu="timing",
 num_cpus=1,
 mem_system="mesi_two_level",
 memory_class="DualChannelDDR3_1600",
-boot_type="init",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=1,
-mem_system="mi_example",
-memory_class="DualChannelDDR3_2133",
-boot_type="init",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=4,
-mem_system="classic",
-memory_class="DualChannelDDR3_2133",
-boot_type="init",
+boot_type="systemd",
 length=constants.long_tag,
 )

@@ -209,14 +183,6 @@
 #length=constants.long_tag,
 # )

-test_boot(
-cpu="atomic",
-num_cpus=4,
-mem_system="classic",
-memory_class="HBM2Stack",
-boot_type="systemd",
-length=constants.long_tag,
-)

  The very-long (Weekly) tests 


--
To view, visit  
https://gem5-review.googlesource.com/c/public/gem5/+/65192?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: I761fca1aa5e111a03183f83d4e326aaea1bdbc3a
Gerrit-Change-Number: 65192
Gerrit-PatchSet: 3
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 X86 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/+/65192?usp=email )



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

tests: Remove long/nightly X86 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 X86 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
remove dto save on testing time.

Change-Id: I761fca1aa5e111a03183f83d4e326aaea1bdbc3a
---
M tests/gem5/x86-boot-tests/test_linux_boot.py
1 file changed, 20 insertions(+), 36 deletions(-)



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

index ecda2ff..7b4c181 100644
--- a/tests/gem5/x86-boot-tests/test_linux_boot.py
+++ b/tests/gem5/x86-boot-tests/test_linux_boot.py
@@ -150,39 +150,13 @@

  The long (Nightly) tests 

-test_boot(
-cpu="atomic",
-num_cpus=1,
-mem_system="classic",
-memory_class="SingleChannelHBM",
-boot_type="init",
-length=constants.long_tag,
-)

 test_boot(
 cpu="timing",
 num_cpus=1,
 mem_system="mesi_two_level",
 memory_class="DualChannelDDR3_1600",
-boot_type="init",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=1,
-mem_system="mi_example",
-memory_class="DualChannelDDR3_2133",
-boot_type="init",
-length=constants.long_tag,
-)
-
-test_boot(
-cpu="timing",
-num_cpus=4,
-mem_system="classic",
-memory_class="DualChannelDDR3_2133",
-boot_type="init",
+boot_type="systemd",
 length=constants.long_tag,
 )

@@ -190,7 +164,7 @@
 cpu="atomic",
 num_cpus=4,
 mem_system="classic",
-memory_class="DualChannelDDR4_2400",
+memory_class="SingleChannelHBM",
 boot_type="systemd",
 length=constants.long_tag,
 )
@@ -209,14 +183,6 @@
 #length=constants.long_tag,
 # )

-test_boot(
-cpu="atomic",
-num_cpus=4,
-mem_system="classic",
-memory_class="HBM2Stack",
-boot_type="systemd",
-length=constants.long_tag,
-)

  The very-long (Weekly) tests 


--
To view, visit  
https://gem5-review.googlesource.com/c/public/gem5/+/65192?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: I761fca1aa5e111a03183f83d4e326aaea1bdbc3a
Gerrit-Change-Number: 65192
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