irqchip: or1k-pic: Undefine mask_ack for level triggered hardware
authorStafford Horne <shorne@gmail.com>
Tue, 14 Jun 2022 23:54:26 +0000 (08:54 +0900)
committerStafford Horne <shorne@gmail.com>
Tue, 28 Jun 2022 08:31:15 +0000 (17:31 +0900)
The mask_ack operation clears the interrupt by writing to the PICSR
register.  This we don't want for level triggered interrupt because
it does not actually clear the interrupt on the source hardware.

This was causing issues in qemu with multi core setups where
interrupts would continue to fire even though they had been cleared in
PICSR.

Just remove the mask_ack operation.

Acked-by: Marc Zyngier <maz@kernel.org>
Signed-off-by: Stafford Horne <shorne@gmail.com>
drivers/irqchip/irq-or1k-pic.c

index 49b47e7..f289ccd 100644 (file)
@@ -66,7 +66,6 @@ static struct or1k_pic_dev or1k_pic_level = {
                .name = "or1k-PIC-level",
                .irq_unmask = or1k_pic_unmask,
                .irq_mask = or1k_pic_mask,
-               .irq_mask_ack = or1k_pic_mask_ack,
        },
        .handle = handle_level_irq,
        .flags = IRQ_LEVEL | IRQ_NOPROBE,