[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and
From: |
Alex Bennée |
Subject: |
[kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants |
Date: |
Wed, 28 Apr 2021 11:18:41 +0100 |
A few of the its-trigger tests rely on IMPDEF behaviour where caches
aren't flushed before invall events. However TCG emulation doesn't
model any invall behaviour and as we can't probe for it we need to be
told. Split the test into a KVM and TCG variant and skip the invall
tests when under TCG.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Shashi Mallela <shashi.mallela@linaro.org>
---
arm/gic.c | 60 +++++++++++++++++++++++++++--------------------
arm/unittests.cfg | 11 ++++++++-
2 files changed, 45 insertions(+), 26 deletions(-)
diff --git a/arm/gic.c b/arm/gic.c
index 98135ef..96a329d 100644
--- a/arm/gic.c
+++ b/arm/gic.c
@@ -36,6 +36,7 @@ static struct gic *gic;
static int acked[NR_CPUS], spurious[NR_CPUS];
static int irq_sender[NR_CPUS], irq_number[NR_CPUS];
static cpumask_t ready;
+static bool under_tcg;
static void nr_cpu_check(int nr)
{
@@ -734,32 +735,38 @@ static void test_its_trigger(void)
/*
* re-enable the LPI but willingly do not call invall
* so the change in config is not taken into account.
- * The LPI should not hit
+ * The LPI should not hit. This does however depend on
+ * implementation defined behaviour - under QEMU TCG emulation
+ * it can quite correctly process the event directly.
*/
- gicv3_lpi_set_config(8195, LPI_PROP_DEFAULT);
- stats_reset();
- cpumask_clear(&mask);
- its_send_int(dev2, 20);
- wait_for_interrupts(&mask);
- report(check_acked(&mask, -1, -1),
- "dev2/eventid=20 still does not trigger any LPI");
-
- /* Now call the invall and check the LPI hits */
- stats_reset();
- cpumask_clear(&mask);
- cpumask_set_cpu(3, &mask);
- its_send_invall(col3);
- wait_for_interrupts(&mask);
- report(check_acked(&mask, 0, 8195),
- "dev2/eventid=20 pending LPI is received");
-
- stats_reset();
- cpumask_clear(&mask);
- cpumask_set_cpu(3, &mask);
- its_send_int(dev2, 20);
- wait_for_interrupts(&mask);
- report(check_acked(&mask, 0, 8195),
- "dev2/eventid=20 now triggers an LPI");
+ if (under_tcg) {
+ report_skip("checking LPI triggers without invall");
+ } else {
+ gicv3_lpi_set_config(8195, LPI_PROP_DEFAULT);
+ stats_reset();
+ cpumask_clear(&mask);
+ its_send_int(dev2, 20);
+ wait_for_interrupts(&mask);
+ report(check_acked(&mask, -1, -1),
+ "dev2/eventid=20 still does not trigger any LPI");
+
+ /* Now call the invall and check the LPI hits */
+ stats_reset();
+ cpumask_clear(&mask);
+ cpumask_set_cpu(3, &mask);
+ its_send_invall(col3);
+ wait_for_interrupts(&mask);
+ report(check_acked(&mask, 0, 8195),
+ "dev2/eventid=20 pending LPI is received");
+
+ stats_reset();
+ cpumask_clear(&mask);
+ cpumask_set_cpu(3, &mask);
+ its_send_int(dev2, 20);
+ wait_for_interrupts(&mask);
+ report(check_acked(&mask, 0, 8195),
+ "dev2/eventid=20 now triggers an LPI");
+ }
report_prefix_pop();
@@ -981,6 +988,9 @@ int main(int argc, char **argv)
if (argc < 2)
report_abort("no test specified");
+ if (argc == 3 && strcmp(argv[2], "tcg") == 0)
+ under_tcg = true;
+
if (strcmp(argv[1], "ipi") == 0) {
report_prefix_push(argv[1]);
nr_cpu_check(2);
diff --git a/arm/unittests.cfg b/arm/unittests.cfg
index f776b66..c72dc34 100644
--- a/arm/unittests.cfg
+++ b/arm/unittests.cfg
@@ -184,13 +184,22 @@ extra_params = -machine gic-version=3 -append
'its-introspection'
groups = its
arch = arm64
-[its-trigger]
+[its-trigger-kvm]
file = gic.flat
smp = $MAX_SMP
+accel = kvm
extra_params = -machine gic-version=3 -append 'its-trigger'
groups = its
arch = arm64
+[its-trigger-tcg]
+file = gic.flat
+smp = $MAX_SMP
+accel = tcg
+extra_params = -machine gic-version=3 -append 'its-trigger tcg'
+groups = its
+arch = arm64
+
[its-migration]
file = gic.flat
smp = $MAX_SMP
--
2.20.1
- [kvm-unit-tests PATCH v1 0/4] enable LPI and ITS for TCG, Alex Bennée, 2021/04/28
- [kvm-unit-tests PATCH v1 2/4] scripts/arch-run: don't use deprecated server/nowait options, Alex Bennée, 2021/04/28
- [kvm-unit-tests PATCH v1 3/4] arm64: enable its-migration tests for TCG, Alex Bennée, 2021/04/28
- [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants,
Alex Bennée <=
- Re: [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants, Marc Zyngier, 2021/04/28
- Re: [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants, Alex Bennée, 2021/04/28
- Re: [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants, Alexandru Elisei, 2021/04/28
- Re: [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants, Marc Zyngier, 2021/04/28
- Re: [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants, Auger Eric, 2021/04/28
- Re: [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants, Alex Bennée, 2021/04/28
- Re: [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants, Alex Bennée, 2021/04/28
- Re: [kvm-unit-tests PATCH v1 1/4] arm64: split its-trigger test into KVM and TCG variants, Marc Zyngier, 2021/04/28
[kvm-unit-tests PATCH v1 4/4] arm64: split its-migrate-unmapped-collection into KVM and TCG variants, Alex Bennée, 2021/04/28