Re: [PATCH v3 3/9] arm: Rely on generic printing of preemption model.

From: Russell King (Oracle)
Date: Wed Feb 12 2025 - 10:59:54 EST


On Wed, Feb 12, 2025 at 02:41:09PM +0100, Sebastian Andrzej Siewior wrote:
> __die() invokes later __show_regs() -> show_regs_print_info() which
> prints the current preemption model.
> Remove it from the initial line.
>
> Cc: Russell King <linux@xxxxxxxxxxxxxxx>

Please either drop this - or correct it as per my r-b below. Thanks.

> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Reviewed-by: Russell King (Oracle) <rmk+kernel@xxxxxxxxxxxxxxx>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>
> ---
> arch/arm/kernel/traps.c | 11 ++---------
> 1 file changed, 2 insertions(+), 9 deletions(-)
>
> diff --git a/arch/arm/kernel/traps.c b/arch/arm/kernel/traps.c
> index 6ea645939573f..afbd2ebe5c39d 100644
> --- a/arch/arm/kernel/traps.c
> +++ b/arch/arm/kernel/traps.c
> @@ -258,13 +258,6 @@ void show_stack(struct task_struct *tsk, unsigned long *sp, const char *loglvl)
> barrier();
> }
>
> -#ifdef CONFIG_PREEMPT
> -#define S_PREEMPT " PREEMPT"
> -#elif defined(CONFIG_PREEMPT_RT)
> -#define S_PREEMPT " PREEMPT_RT"
> -#else
> -#define S_PREEMPT ""
> -#endif
> #ifdef CONFIG_SMP
> #define S_SMP " SMP"
> #else
> @@ -282,8 +275,8 @@ static int __die(const char *str, int err, struct pt_regs *regs)
> static int die_counter;
> int ret;
>
> - pr_emerg("Internal error: %s: %x [#%d]" S_PREEMPT S_SMP S_ISA "\n",
> - str, err, ++die_counter);
> + pr_emerg("Internal error: %s: %x [#%d]" S_SMP S_ISA "\n",
> + str, err, ++die_counter);
>
> /* trap and error numbers are mostly meaningless on ARM */
> ret = notify_die(DIE_OOPS, str, regs, err, tsk->thread.trap_no, SIGSEGV);
> --
> 2.47.2
>
>

--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!