Re: [PATCH] hw/intc/ioapic: Update KVM routes before redelivering IRQ, on RTE update

[David Woodhouse]

On 5 March 2023 22:36:18 GMT, Peter Xu <peterx@redhat.com> wrote:
>On Sun, Mar 05, 2023 at 06:43:42PM +0000, 
>> ---
>> Alternative fixes might have been just to remove the part in
>> ioapic_service() which delivers the IRQ via kvm_set_irq() because
>> surely delivering as MSI ought to work just fine anyway in all cases?
>> That code lacks a comment justifying its existence.
>
>Didn't check all details, but AFAIU there're still some different paths
>triggered so at least it looks still clean to use the path it's for.
>
>E.g., I think if someone traces kvm_set_irq() in kernel this specific irq
>triggered right after unmasking might seem to be missed misterously (but
>actually it was not).

Hm, not sure that's a reason we care about. The I/OAPIC is purely a device to 
turn line interrupts into MSIs. Which these days need to be translated by IOMMU 
interrupt remapping device models in userspace. I don't think a user has any 
valid reason to expect that the kernel will even know about any GSIs with any 
specific numbers. Tracing on that in the kernel would making some dodgy 
assumptions.