Re: [tip: irq/msi] PCI: dwc: Enable MSI affinity support

[Radu Rendec <rrendec@redhat.com>]

Hi Jon,

On Tue, 2026-01-06 at 09:53 +0000, Jon Hunter wrote:
> On 15/12/2025 21:34, tip-bot2 for Radu Rendec wrote:
> > The following commit has been merged into the irq/msi branch of tip:
> > 
> > Commit-ID:     eaf290c404f7c39f23292e9ce83b8b5b51ab598a
> > Gitweb:        https://git.kernel.org/tip/eaf290c404f7c39f23292e9ce83b8b5b51ab598a
> > Author:        Radu Rendec <rrendec@redhat.com>
> > AuthorDate:    Fri, 28 Nov 2025 16:20:55 -05:00
> > Committer:     Thomas Gleixner <tglx@linutronix.de>
> > CommitterDate: Mon, 15 Dec 2025 22:30:48 +01:00
> > 
> > PCI: dwc: Enable MSI affinity support
> > 
> > Leverage the interrupt redirection infrastructure to enable CPU affinity
> > support for MSI interrupts. Since the parent interrupt affinity cannot
> > be changed, affinity control for the child interrupt (MSI) is achieved
> > by redirecting the handler to run in IRQ work context on the target CPU.
> > 
> > This patch was originally prepared by Thomas Gleixner (see Link tag below)
> > in a patch series that was never submitted as is, and only parts of that
> > series have made it upstream so far.
> > 
> > Originally-by: Thomas Gleixner <tglx@linutronix.de>
> > Signed-off-by: Radu Rendec <rrendec@redhat.com>
> > Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
> > Link: https://lore.kernel.org/linux-pci/878qpg4o4t.ffs@tglx/
> > Link: https://patch.msgid.link/20251128212055.1409093-4-rrendec@redhat.com
> 
> 
> With next-20260105 I am observing the following warning on the Tegra194 
> Jetson AGX platform ...
> 
>   WARNING KERN genirq: irq_chip DW-PCI-MSI-0001:01:00.0 did not update
>    eff. affinity mask of irq 171
> 
> Bisect is point to this commit. This platform is using the driver 
> drivers/pci/controller/dwc/pcie-tegra194.c. Is there some default 
> affinity that we should be setting to avoid this warning?

Before that patch, affinity control wasn't even possible for PCI MSIs
exposed by the dw_pci drivers. Without having looked at the code yet,
I suspect it's just because now that affinity control is enabled,
something tries to use it.

I don't think you should set some default affinity. By default, the PCI
MSIs should be affine to all available CPUs, and that warning shouldn't
happen in the first place. Let me test on Jetson AGX and see what's
going on. I'll update the thread with my findings, hopefully later
today.

-- 
Thanks,
Radu