[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 5/6] scripts/device-crash-test: Add a parameter to run with TC
From: |
Alex Bennée |
Subject: |
[PATCH v4 5/6] scripts/device-crash-test: Add a parameter to run with TCG only |
Date: |
Mon, 17 Apr 2023 14:43:20 +0100 |
From: Thomas Huth <thuth@redhat.com>
We're currently facing the problem that the device-crash-test script
runs twice as long in the CI when a runner supports KVM - which sometimes
results in a timeout of the CI job. To get a more deterministic runtime
here, add an option to the script that allows to run it with TCG only.
Reported-by: Eldon Stegall <eldon-qemu@eldondev.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20230414145845.456145-3-thuth@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.gitlab-ci.d/buildtest.yml | 2 +-
scripts/device-crash-test | 4 +++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
index 333eea9dd3..bb3650a51c 100644
--- a/.gitlab-ci.d/buildtest.yml
+++ b/.gitlab-ci.d/buildtest.yml
@@ -103,7 +103,7 @@ crash-test-debian:
script:
- cd build
- make NINJA=":" check-venv
- - tests/venv/bin/python3 scripts/device-crash-test -q ./qemu-system-i386
+ - tests/venv/bin/python3 scripts/device-crash-test -q --tcg-only
./qemu-system-i386
build-system-fedora:
extends:
diff --git a/scripts/device-crash-test b/scripts/device-crash-test
index 73bcb98693..b74d887331 100755
--- a/scripts/device-crash-test
+++ b/scripts/device-crash-test
@@ -397,7 +397,7 @@ def binariesToTest(args, testcase):
def accelsToTest(args, testcase):
- if getBinaryInfo(args, testcase['binary']).kvm_available:
+ if getBinaryInfo(args, testcase['binary']).kvm_available and not
args.tcg_only:
yield 'kvm'
yield 'tcg'
@@ -510,6 +510,8 @@ def main():
help="Full mode: test cases that are expected to fail")
parser.add_argument('--strict', action='store_true', dest='strict',
help="Treat all warnings as fatal")
+ parser.add_argument('--tcg-only', action='store_true', dest='tcg_only',
+ help="Only test with TCG accelerator")
parser.add_argument('qemu', nargs='*', metavar='QEMU',
help='QEMU binary to run')
args = parser.parse_args()
--
2.39.2
Re: [PATCH v4 1/6] tests/requirements.txt: bump up avocado-framework version to 101.0, Thomas Huth, 2023/04/18
Re: [PATCH v4 1/6] tests/requirements.txt: bump up avocado-framework version to 101.0, Cleber Rosa, 2023/04/20
Re: [PATCH v4 1/6] tests/requirements.txt: bump up avocado-framework version to 101.0, Thomas Huth, 2023/04/21
[PATCH v4 6/6] qemu-options: finesse the recommendations around -blockdev, Alex Bennée, 2023/04/17
[PATCH v4 5/6] scripts/device-crash-test: Add a parameter to run with TCG only,
Alex Bennée <=
[PATCH v4 2/6] tests/avocado: use the new snapshots for testing, Alex Bennée, 2023/04/17
[PATCH v4 3/6] tests/avocado: Add set of boot tests on SBSA-ref, Alex Bennée, 2023/04/17
Re: [PATCH v4 0/6] testing/next: avocado, docs, gitlab, John Snow, 2023/04/17