Re: Purpose of translate MSI interrupt into INTx for guest passthrough

[Jason Andryuk <jandryuk@xxxxxxxxx>]

On Tue, Jan 12, 2021 at 9:25 AM Roger Pau Monné <roger.pau@xxxxxxxxxx> wrote:
>
> Dropping Qing He as this address bounces.
>
> On Tue, Jan 12, 2021 at 03:10:57PM +0100, Roger Pau Monné wrote:
> > Hello,
> >
> > While trying to do some cleanup of the Xen interrupt support for pci
> > pass though I came across the MSI to INTx translation that Xen is in
> > theory capable of performing (ie: use a physical MSI interrupt source
> > and inject that as an INTx to a guest).
> >
> > AFAICT such functionality is not wired up to the toolstack, so it's
> > hard to tell what's the indented purpose, or whether it has seen any
> > usage.
>
> So apparently it is wired up to the toolstack for qemu-traditional,
> albeit it's disabled by default. There's some documentation in
> xl-pci-configuration:
>
> "When enabled, MSI-INTx translation will always enable MSI on the PCI
> device regardless of whether the guest uses INTx or MSI."
>
> So the main purpose seem to be to always use the MSI interrupt source
> regardless of whether the guest is using INTx or MSI. Maybe the
> purpose was to workaround some bugs when using INTx? Or buggy devices
> with INTx interrupts?
>
> qemu-upstream doesn't seem to support it anymore, so I would still
> like to remove it if we get consensus.

The cover letter from
http://old-list-archives.xenproject.org/archives/html/xen-devel/2009-01/msg00228.html
"""
This patchset enables MSI-INTx interrupt translation for HVM
domains. The intention of the patch is to use MSI as the physical
interrupt mechanism for passthrough device as much as possible,
thus reducing the pirq sharing among domains.

When MSI is globally enabled, if the device has the MSI capability
but doesn't used by the guest, hypervisor will try to user MSI as
the underlying pirq and inject translated INTx irq to guest
vioapic. When guest itself enabled MSI or MSI-X, the translation
is automatically turned off.

Add a config file option to disable/enable this feature. Also, in
order to allow the user to override the option per device, a
per-device option mechanism is implemented and an MSI-INTx option
is added
"""

It seems like it could be a good idea, but I don't know if it presents
compatibility issues when actually used.

As you say, it's not supported by qemu-upstream, so maybe it should
just be dropped.

Regards,
Jason