Re: [EXTERNAL] Re: [PATCH] x86/ioapic: Don't return 0 as valid virq

From: Borislav Petkov
Date: Mon Mar 13 2023 - 07:20:04 EST


On Mon, Mar 13, 2023 at 03:29:32AM +0000, Saurabh Singh Sengar wrote:
> To be specific in our system which is a guest VM we don't need IO-APIC and hence
> there is no device tree node for it. It is observed that we get irq 0 assigned to PCI-MSI.

This should be added to your commit message: what guest VM is that and
why should the kernel support it.

Why doesn't it need an IO-APIC and why does the current code need to be
changed just for your guest VM?

What else needs to be changed so that your VM works?

Where is that VM's documentation and why can't that VM be fixed *not* to
need kernel changes? IOW, why can't that VM emulate an IO-APIC like the
others do...

--
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette