Re: [PATCH v2 0/9] preempt: Add a generic function to return the preemption string.
From: Peter Zijlstra
Date: Mon Feb 03 2025 - 09:29:38 EST
On Mon, Feb 03, 2025 at 03:16:23PM +0100, Sebastian Andrzej Siewior wrote:
> The individual architectures often add the preemption model to the begin
> of the backtrace. This is the case on X86 or ARM64 for the "die" case
> but not for regular warning. With the addition of DYNAMIC_PREEMPT for
> PREEMPT_RT we end up with CONFIG_PREEMPT and CONFIG_PREEMPT_RT set
> simultaneously. That means that everyone who tried to add that piece of
> information gets it wrong for PREEMPT_RT because PREEMPT is checked
> first.
>
> This series adds a generic helper which is used by
> dump_stack_print_info() and other architectures which don't use the
> function and provide this information on their own.
>
> RFC…v2 https://lore.kernel.org/all/20241206113431.Q-VXMlru@xxxxxxxxxxxxx/
> - Split into individual patches.
> - Implement preempt_model_str() based on feedback by Peter Zijlstra
> and Steven Rostedt.
>
> Sebastian
>
> Sebastian Andrzej Siewior (9):
> sched: Add a generic function to return the preemption string.
> lib/dump_stack: Use preempt_model_str().
> arm: Rely on generic printing of preemption model.
> arm64: Rely on generic printing of preemption model.
> powerpc: Use preempt_model_str().
> s390: Rely on generic printing of preemption model.
> x86: Rely on generic printing of preemption model.
> xtensa: Use preempt_model_str().
> tracing: Use preempt_model_str().
Works for me; how do we want to go about merging this?
I can take it through the sched tree I suppose, once the various arch
people have blessed it.