Re: [PATCH] softirq: Print the function name

["enlin.mu" <enlin.mu@linux.dev>]

On 2026/1/5 17:41, Enlin Mu wrote:
> Adjust the printing format from the function address to
> the function name.
> 
> Signed-off-by: Enlin Mu <enlin.mu@unisoc.com>
> Signed-off-by: Enlin Mu <enlin.mu@linux.dev>
> ---
>   kernel/softirq.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/kernel/softirq.c b/kernel/softirq.c
> index 77198911b8dd..68ef18616fb9 100644
> --- a/kernel/softirq.c
> +++ b/kernel/softirq.c
> @@ -622,7 +622,7 @@ static void handle_softirqs(bool ksirqd)
>   		h->action();
>   		trace_softirq_exit(vec_nr);
>   		if (unlikely(prev_count != preempt_count())) {
> -			pr_err("huh, entered softirq %u %s %p with preempt_count %08x, exited with %08x?\n",
> +			pr_err("huh, entered softirq %u %s %pS with preempt_count %08x, exited with %08x?\n",
>   			       vec_nr, softirq_to_name[vec_nr], h->action,
>   			       prev_count, preempt_count());

Hi All

My test results are as follows:

before this patch:

[   49.919574] softirq: huh, entered softirq 1 TIMER 00000000bb45ef3a 
with preempt_count 00000101, exited with 00000101?


after this patch:

[   49.919570] softirq: huh, entered softirq 1 TIMER 
run_timer_softirq+0x0/0x108 with preempt_count 00000101, exited with 
00000102?

>   			preempt_count_set(prev_count);