Giacomo Travaglini has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/31219 )
Change subject: tests: Add realview64-kvm.py test to quick regressions
......................................................................
tests: Add realview64-kvm.py test to quick regressions
By using the valid_host parameter we can make sure the test is
run on a aarch64 host only
Signed-off-by: Giacomo Travaglini <[email protected]>
Change-Id: I3cdb35967e85377f26adf73ad147cb2479162ca1
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/31219
Tested-by: kokoro <[email protected]>
Reviewed-by: Jason Lowe-Power <[email protected]>
Reviewed-by: Andreas Sandberg <[email protected]>
Maintainer: Jason Lowe-Power <[email protected]>
---
M tests/gem5/fs/linux/arm/test.py
1 file changed, 20 insertions(+), 1 deletion(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
Andreas Sandberg: Looks good to me, approved
kokoro: Regressions pass
diff --git a/tests/gem5/fs/linux/arm/test.py
b/tests/gem5/fs/linux/arm/test.py
index 482dd1d..b02d1af 100644
--- a/tests/gem5/fs/linux/arm/test.py
+++ b/tests/gem5/fs/linux/arm/test.py
@@ -41,6 +41,10 @@
from testlib import *
+arm_fs_kvm_tests = [
+ 'realview64-kvm',
+]
+
arm_fs_quick_tests = [
'realview64-simple-atomic',
'realview64-simple-atomic-dual',
@@ -49,7 +53,7 @@
'realview64-simple-timing-dual',
'realview64-switcheroo-atomic',
'realview64-switcheroo-timing',
-]
+] + arm_fs_kvm_tests
arm_fs_long_tests = [
'realview-simple-atomic',
@@ -94,7 +98,21 @@
path = joinpath(config.bin_path, 'arm')
arm_fs_binaries = DownloadedArchive(url, path, tarball)
+def support_kvm():
+ return os.access("/dev/kvm", os.R_OK | os.W_OK)
+
for name in arm_fs_quick_tests:
+ if name in arm_fs_kvm_tests:
+ # The current host might not be supporting KVM
+ # Skip the test if that's the case
+ if not support_kvm():
+ continue
+
+ # Run KVM test if we are on an arm host only
+ valid_hosts = (constants.host_arm_tag,)
+ else:
+ valid_hosts = constants.supported_hosts
+
args = [
joinpath(config.base_dir, 'tests', 'gem5', 'configs', name
+ '.py'),
path,
@@ -107,6 +125,7 @@
config_args=args,
valid_isas=(constants.arm_tag,),
length=constants.quick_tag,
+ valid_hosts=valid_hosts,
fixtures=(arm_fs_binaries,)
)
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/31219
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: I3cdb35967e85377f26adf73ad147cb2479162ca1
Gerrit-Change-Number: 31219
Gerrit-PatchSet: 11
Gerrit-Owner: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[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
