Re: Patch "arm64: entry: factor irq triage logic into macros" has been added to the 5.12-stable tree

* Re: Patch "arm64: entry: factor irq triage logic into macros" has been added to the 5.12-stable tree
       [not found] <20210515021826.35E98613F2@mail.kernel.org>
@ 2021-05-15 10:40 ` Marc Zyngier
  2021-05-15 12:36   ` Sasha Levin
  2021-05-15 12:37   ` Sasha Levin
  0 siblings, 2 replies; 4+ messages in thread
From: Marc Zyngier @ 2021-05-15 10:40 UTC (permalink / raw)
  To: Sasha Levin; +Cc: stable-commits, stable, Mark Rutland

Hi Sasha,

On Sat, 15 May 2021 03:18:25 +0100,
Sasha Levin <sashal@kernel.org> wrote:
> 
> This is a note to let you know that I've just added the patch titled
> 
>     arm64: entry: factor irq triage logic into macros
> 
> to the 5.12-stable tree which can be found at:
>     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> 
> The filename of the patch is:
>      arm64-entry-factor-irq-triage-logic-into-macros.patch
> and it can be found in the queue-5.12 subdirectory.
> 
> If you, or anyone else, feels it should not be added to the stable tree,
> please let <stable@vger.kernel.org> know about it.
>
> 
> 
> commit ec301e4d131aad1648e28305c8d02ae8265a50d7
> Author: Marc Zyngier <maz@kernel.org>
> Date:   Mon Mar 15 11:56:27 2021 +0000
> 
>     arm64: entry: factor irq triage logic into macros
>     
>     [ Upstream commit 9eb563cdabe1d583c262042d5d44cc256f644543 ]
>     
>     In subsequent patches we'll allow an FIQ handler to be registered, and
>     FIQ exceptions will need to be triaged very similarly to IRQ exceptions.
>     So that we can reuse the existing logic, this patch factors the IRQ
>     triage logic out into macros that can be reused for FIQ.
>     
>     The macros are named to follow the elX_foo_handler scheme used by the C
>     exception handlers. For consistency with other top-level exception
>     handlers, the kernel_entry/kernel_exit logic is not moved into the
>     macros. As FIQ will use a different C handler, this handler name is
>     provided as an argument to the macros.
>     
>     There should be no functional change as a result of this patch.
>     
>     Signed-off-by: Marc Zyngier <maz@kernel.org>
>     [Mark: rework macros, commit message, rebase before DAIF rework]
>     Signed-off-by: Mark Rutland <mark.rutland@arm.com>
>     Tested-by: Hector Martin <marcan@marcan.st>
>     Cc: James Morse <james.morse@arm.com>
>     Cc: Thomas Gleixner <tglx@linutronix.de>
>     Cc: Will Deacon <will@kernel.org>
>     Acked-by: Will Deacon <will@kernel.org>
>     Link: https://lore.kernel.org/r/20210315115629.57191-5-mark.rutland@arm.com
>     Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
>     Signed-off-by: Sasha Levin <sashal@kernel.org>

I don't think there is any reason for backporting this patch at this
stage. It isn't a fix, and we don't plan to support the feature it
subsequently enables in anything older than 5.13.

Unless there is a another pressing reason for adding this patch, I
suggest it is dropped from 5.10, 5.11 and 5.12 stable branches.

Thanks,

	M.

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

^ permalink raw reply	[flat|nested] 4+ messages in thread