linux-kernel - Re: [patch 1/1] softirq: improve preempt count error message

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20101005074554.GB23608@elte.hu>
Date:	Tue, 5 Oct 2010 09:45:54 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	akpm@...ux-foundation.org
Cc:	tglx@...utronix.de, jslaby@...e.cz, linux-kernel@...r.kernel.org,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>
Subject: Re: [patch 1/1] softirq: improve preempt count error message


* akpm@...ux-foundation.org <akpm@...ux-foundation.org> wrote:

> From: Jiri Slaby <jslaby@...e.cz>
> 
> * add a space after the action
> * print function name instead of address of action (%p->%pf)
> 
> Signed-off-by: Jiri Slaby <jslaby@...e.cz>
> Cc: Ingo Molnar <mingo@...e.hu>
> Cc: Thomas Gleixner <tglx@...utronix.de>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
> 
>  kernel/softirq.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff -puN kernel/softirq.c~softirq-improve-preempt-count-error-message kernel/softirq.c
> --- a/kernel/softirq.c~softirq-improve-preempt-count-error-message
> +++ a/kernel/softirq.c
> @@ -219,8 +219,8 @@ restart:
>  			h->action(h);
>  			trace_softirq_exit(h, softirq_vec);
>  			if (unlikely(prev_count != preempt_count())) {
> -				printk(KERN_ERR "huh, entered softirq %td %s %p"
> -				       "with preempt_count %08x,"
> +				printk(KERN_ERR "huh, entered softirq %td %s "
> +				       "%pf with preempt_count %08x,"
>  				       " exited with %08x?\n", h - softirq_vec,
>  				       softirq_to_name[h - softirq_vec],
>  				       h->action, prev_count, preempt_count());

Please fix the cause of the bug as well: i think this whole block of 
debug checks should move into a softirq_debug_check(h, prev_count) 
helper inline function - that would clean up the insane printk line 
which is currently spread out on 5 lines ...

It would also make __do_softirq() more readable.

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/