hw/intc/arm_gicv3_cpuif: Make GIC maintenance interrupts work
authorPeter Maydell <peter.maydell@linaro.org>
Mon, 2 Nov 2020 16:52:17 +0000 (16:52 +0000)
committerPeter Maydell <peter.maydell@linaro.org>
Mon, 2 Nov 2020 16:52:17 +0000 (16:52 +0000)
commitc20a135a7a14d846699b0cfc9f74bcc2a226bee6
tree9120ea53f221fddd9bac82e4e1a73ab53838402f
parent76346b6264a9b01979829a5d4366c3799e2dd15a
hw/intc/arm_gicv3_cpuif: Make GIC maintenance interrupts work

In gicv3_init_cpuif() we copy the ARMCPU gicv3_maintenance_interrupt
into the GICv3CPUState struct's maintenance_irq field.  This will
only work if the board happens to have already wired up the CPU
maintenance IRQ before the GIC was realized.  Unfortunately this is
not the case for the 'virt' board, and so the value that gets copied
is NULL (since a qemu_irq is really a pointer to an IRQState struct
under the hood).  The effect is that the CPU interface code never
actually raises the maintenance interrupt line.

Instead, since the GICv3CPUState has a pointer to the CPUState, make
the dereference at the point where we want to raise the interrupt, to
avoid an implicit requirement on board code to wire things up in a
particular order.

Reported-by: Jose Martins <josemartins90@gmail.com>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: 20201009153904.28529-1-peter.maydell@linaro.org
Reviewed-by: Luc Michel <luc@lmichel.fr>
hw/intc/arm_gicv3_cpuif.c
include/hw/intc/arm_gicv3_common.h