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

Change subject: tests: Move MI_Example tests to Long/Nightly
......................................................................

tests: Move MI_Example tests to Long/Nightly

Two tests, one in test_kvm_fork_run.py and another in
test_kvm_cpu_switch.py, were trying to run the MI_Example protocol as
part of the quick/kokoro run. MI_Example requires the building of X86,
though we try to use GCN3_X86 exclusively to avoid compiling an
additional target. As such, these MI_Example tests have been moved to
the long/nightly run.

Change-Id: I3c196e7e336148a1b7a124b5810348a2e587fe24
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/52563
Reviewed-by: Gabe Black <gabe.bl...@gmail.com>
Reviewed-by: Jason Lowe-Power <power...@gmail.com>
Maintainer: Jason Lowe-Power <power...@gmail.com>
Tested-by: kokoro <noreply+kok...@google.com>
---
M tests/gem5/kvm-fork-tests/test_kvm_fork_run.py
M tests/gem5/kvm-switch-tests/test_kvm_cpu_switch.py
2 files changed, 27 insertions(+), 6 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  Gabe Black: Looks good to me, but someone else must approve
  kokoro: Regressions pass




diff --git a/tests/gem5/kvm-fork-tests/test_kvm_fork_run.py b/tests/gem5/kvm-fork-tests/test_kvm_fork_run.py
index fb5ff2b..bd7fd6a 100644
--- a/tests/gem5/kvm-fork-tests/test_kvm_fork_run.py
+++ b/tests/gem5/kvm-fork-tests/test_kvm_fork_run.py
@@ -107,15 +107,15 @@
cpu="timing", num_cpus=4, mem_system="classic", length=constants.quick_tag
 )

+### The long (nightly) tests ####
+
 test_kvm_fork_run(
     cpu="timing",
     num_cpus=4,
     mem_system="mi_example",
-    length=constants.quick_tag,
+    length=constants.long_tag,
 )

-### The long (nightly) tests ####
-
 test_kvm_fork_run(
     cpu="timing",
     num_cpus=1,
diff --git a/tests/gem5/kvm-switch-tests/test_kvm_cpu_switch.py b/tests/gem5/kvm-switch-tests/test_kvm_cpu_switch.py
index 718189c..0ca32db 100644
--- a/tests/gem5/kvm-switch-tests/test_kvm_cpu_switch.py
+++ b/tests/gem5/kvm-switch-tests/test_kvm_cpu_switch.py
@@ -105,15 +105,15 @@
cpu="timing", num_cpus=4, mem_system="classic", length=constants.quick_tag
 )

+### The long (nightly) tests ####
+
 test_kvm_switch(
     cpu="timing",
     num_cpus=4,
     mem_system="mi_example",
-    length=constants.quick_tag,
+    length=constants.long_tag,
 )

-### The long (nightly) tests ####
-
 test_kvm_switch(
     cpu="timing",
     num_cpus=1,

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/52563
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: I3c196e7e336148a1b7a124b5810348a2e587fe24
Gerrit-Change-Number: 52563
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Gabe Black <gabe.bl...@gmail.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
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to