Commit 8b7888c5 authored by Marc Zyngier's avatar Marc Zyngier Committed by Oliver Upton
Browse files

KVM: arm64: selftests: vgic_irq: Change configuration before enabling interrupt 



The architecture is pretty clear that changing the configuration of
an enable interrupt is not OK. It doesn't really matter here, but
doing the right thing is not more expensive.

Tested-by: default avatarFuad Tabba <tabba@google.com>
Signed-off-by: default avatarMarc Zyngier <maz@kernel.org>
Tested-by: default avatarMark Brown <broonie@kernel.org>
Link: https://msgid.link/20251120172540.2267180-44-maz@kernel.org


Signed-off-by: default avatarOliver Upton <oupton@kernel.org>
parent 27392612

tools/testing/selftests/kvm/arm64/vgic_irq.c

+3 −3
Original line number Diff line number Diff line
@@ -473,12 +473,12 @@ static void guest_code(struct test_args *args) 


	gic_init(GIC_V3, 1);



	for (i = 0; i < nr_irqs; i++)

		gic_irq_enable(i);



	for (i = MIN_SPI; i < nr_irqs; i++)

		gic_irq_set_config(i, !level_sensitive);



	for (i = 0; i < nr_irqs; i++)

		gic_irq_enable(i);



	gic_set_eoi_split(args->eoi_split);



	reset_priorities(args);