tests: Fix 'valid_hosts' field for KVM tests
In some cases where we wish to use X86 KVM the 'valid_hosts' field was set to 'constants.supported_hosts'. This can cause problems as the only valid host for an X86 KVM test is X86. This patch fixes this. This fix will likely fix some flakey errors observed in the Nightly tests. Change-Id: Id0a906ca6602c0358ae632cce314769807bf6e07 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/58970 Reviewed-by: Jason Lowe-Power <power.jg@gmail.com> Maintainer: Jason Lowe-Power <power.jg@gmail.com> Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
committed by
Bobby Bruce
parent
6d8a5ad2ad
commit
35f7008602
@@ -68,7 +68,7 @@ if os.access("/dev/kvm", mode=os.R_OK | os.W_OK):
|
||||
),
|
||||
config_args=[],
|
||||
valid_isas=(constants.x86_tag,),
|
||||
valid_hosts=constants.supported_hosts,
|
||||
valid_hosts=(constants.host_x86_64_tag,),
|
||||
length=constants.long_tag,
|
||||
uses_kvm=True,
|
||||
)
|
||||
@@ -107,7 +107,7 @@ if os.access("/dev/kvm", mode=os.R_OK | os.W_OK):
|
||||
config_args=["--benchmark","blackscholes","--size","simsmall"],
|
||||
valid_isas=(constants.x86_tag,),
|
||||
protocol="MESI_Two_Level",
|
||||
valid_hosts=constants.supported_hosts,
|
||||
valid_hosts=(constants.host_x86_64_tag,),
|
||||
length=constants.long_tag,
|
||||
uses_kvm=True,
|
||||
)
|
||||
@@ -135,7 +135,7 @@ if os.access("/dev/kvm", mode=os.R_OK | os.W_OK):
|
||||
],
|
||||
valid_isas=(constants.x86_tag,),
|
||||
protocol="MESI_Two_Level",
|
||||
valid_hosts=constants.supported_hosts,
|
||||
valid_hosts=(constants.host_x86_64_tag,),
|
||||
length=constants.long_tag,
|
||||
uses_kvm=True,
|
||||
)
|
||||
@@ -157,7 +157,7 @@ if os.access("/dev/kvm", mode=os.R_OK | os.W_OK):
|
||||
config_args=["--benchmark","bfs","--synthetic","1","--size","1"],
|
||||
valid_isas=(constants.x86_tag,),
|
||||
protocol="MESI_Two_Level",
|
||||
valid_hosts=constants.supported_hosts,
|
||||
valid_hosts=(constants.host_x86_64_tag,),
|
||||
length=constants.long_tag,
|
||||
uses_kvm=True,
|
||||
)
|
||||
|
||||
@@ -86,7 +86,7 @@ def test_kvm_fork_run(cpu: str, num_cpus: int, mem_system: str, length: str):
|
||||
"--kernel-args=''",
|
||||
],
|
||||
valid_isas=(isa_to_use,),
|
||||
valid_hosts=constants.supported_hosts,
|
||||
valid_hosts=(constants.host_x86_64_tag,),
|
||||
protocol=protocol_to_use,
|
||||
length=length,
|
||||
uses_kvm=True,
|
||||
|
||||
@@ -84,7 +84,7 @@ def test_kvm_switch(cpu: str, num_cpus: int, mem_system: str, length: str):
|
||||
"--kernel-args=''",
|
||||
],
|
||||
valid_isas=(isa_to_use,),
|
||||
valid_hosts=constants.supported_hosts,
|
||||
valid_hosts=(constants.host_x86_64_tag,),
|
||||
protocol=protocol_to_use,
|
||||
length=length,
|
||||
uses_kvm=True,
|
||||
|
||||
@@ -86,7 +86,7 @@ def test_parsec(
|
||||
resource_path,
|
||||
],
|
||||
valid_isas=(constants.x86_tag,),
|
||||
valid_hosts=constants.supported_hosts,
|
||||
valid_hosts=(constants.host_x86_64_tag,),
|
||||
length=length,
|
||||
uses_kvm=True,
|
||||
)
|
||||
|
||||
Reference in New Issue
Block a user