• Marc Zyngier's avatar
    KVM: arm/arm64: vgic: Disallow Active+Pending for level interrupts · 67b5b673
    Marc Zyngier authored
    It was recently reported that VFIO mediated devices, and anything
    that VFIO exposes as level interrupts, do no strictly follow the
    expected logic of such interrupts as it only lowers the input
    line when the guest has EOId the interrupt at the GIC level, rather
    than when it Acked the interrupt at the device level.
    
    THe GIC's Active+Pending state is fundamentally incompatible with
    this behaviour, as it prevents KVM from observing the EOI, and in
    turn results in VFIO never dropping the line. This results in an
    interrupt storm in the guest, which it really never expected.
    
    As we cannot really change VFIO to follow the strict rules of level
    signalling, let's forbid the A+P state altogether, as it is in the
    end only an optimization. It ensures that we will transition via
    an invalid state, which we can use to notify VFIO of the EOI.
    Reviewed-by: default avatarEric Auger <eric.auger@redhat.com>
    Tested-by: default avatarEric Auger <eric.auger@redhat.com>
    Tested-by: default avatarShunyong Yang <shunyong.yang@hxt-semitech.com>
    Signed-off-by: default avatarMarc Zyngier <marc.zyngier@arm.com>
    67b5b673
vgic-v2.c 13.2 KB