(updated some email addresses in cc, including my own) On Sat, 15 Feb 2020 at 01:54, Alan Mikhak <alan.mikhak@xxxxxxxxxx> wrote: > > Hi.. > > What is the right approach for adding MSI support for the generic > Linux PCI host driver? > > I came across this patch which seems to address a similar > situation. It seems to have been dropped in v3 of the patchset > with the explanation "drop MSI patch [for now], since it > turns out we may not need it". > > [PATCH 2/3] pci: designware: add separate driver for the MSI part of the RC > https://lore.kernel.org/linux-pci/20170821192907.8695-3-ard.biesheuvel@xxxxxxxxxx/ > > [PATCH v2 2/3] pci: designware: add separate driver for the MSI part of the RC > https://lore.kernel.org/linux-pci/20170824184321.19432-3-ard.biesheuvel@xxxxxxxxxx/ > > [PATCH v3 0/2] pci: add support for firmware initialized designware RCs > https://lore.kernel.org/linux-pci/20170828180437.2646-1-ard.biesheuvel@xxxxxxxxxx/ > For the platform in question, it turned out that we could use the MSI block of the core's GIC interrupt controller directly, which is a much better solution. In general, turning MSIs into wired interrupts is not a great idea, since the whole point of MSIs is that they are sufficiently similar to other DMA transactions to ensure that the interrupt won't arrive before the related memory transactions have completed. If your interrupt controller does not have this capability, then yes, you are stuck with this little widget that decodes an inbound write to a magic address and turns it into a wired interrupt. I'll leave it up to the Synopsys folks to comment on whether this feature is generic enough to describe it like this, but if so, I think it still makes sense to model it this way rather than fold it into the RC driver and description.
