lkml.org 
[lkml]   [2022]   [Dec]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH AUTOSEL 4.14 3/4] powerpc/msi: Fix deassociation of MSI descriptors
On Tue, 27 Dec 2022 20:36:08 +0000,
Sasha Levin <sashal@kernel.org> wrote:
>
> From: Marc Zyngier <maz@kernel.org>
>
> [ Upstream commit 4545c6a3d6ba71747eaa984c338ddd745e56e23f ]
>
> Since 2f2940d16823 ("genirq/msi: Remove filter from
> msi_free_descs_free_range()"), the core MSI code relies on the
> msi_desc->irq field to have been cleared before the descriptor
> can be freed, as it indicates that there is no association with
> a device anymore.
>
> The irq domain code provides this guarantee, and so does s390,
> which is one of the two architectures not using irq domains for
> MSIs.
>
> Powerpc, however, is missing this particular requirements,
> leading in a splat and leaked MSI descriptors.
>
> Adding the now required irq reset to the handful of powerpc backends
> that implement MSIs fixes that particular problem.
>
> Reported-by: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Marc Zyngier <maz@kernel.org>
> Link: https://lore.kernel.org/r/70dab88e-6119-0c12-7c6a-61bcbe239f66@roeck-us.net
> Signed-off-by: Sasha Levin <sashal@kernel.org>
> ---
> arch/powerpc/platforms/4xx/hsta_msi.c | 1 +
> arch/powerpc/platforms/cell/axon_msi.c | 1 +
> arch/powerpc/platforms/pasemi/msi.c | 1 +
> arch/powerpc/sysdev/fsl_msi.c | 1 +
> arch/powerpc/sysdev/mpic_u3msi.c | 1 +
> 5 files changed, 5 insertions(+)

Please drop this patch from all stable branches. It isn't needed
before 6.2, and doesn't fix anything on its own as nobody uses this
structure after this point.

M.

--
Without deviation from the norm, progress is not possible.

\
 
 \ /
  Last update: 2023-03-26 23:22    [W:0.038 / U:0.816 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site