linux-kernel - Re: [PATCH] let WARN_ON() output the condition

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-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20061206090715.GA30931@elte.hu>
Date:	Wed, 6 Dec 2006 10:07:15 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	Jiri Kosina <jikos@...os.cz>
Cc:	Andrew Morton <akpm@...l.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] let WARN_ON() output the condition


* Jiri Kosina <jikos@...os.cz> wrote:

> > but i agree with you in theory that your proposed output is better, but 
> > the side-effect issue is a killer i think. Could you try to rework it to 
> > not evaluate the condition twice and to make it dependent on 
> > CONFIG_DEBUG_BUGVERBOSE? You can avoid the evaluation side-effect issue 
> > by doing something like:
> > 	int __c = (c);							\
> >                                                                         \
> >         if (unlikely(__c)) {                                            \
> >                 if (debug_locks_off())                                  \
> >                         WARN_ON(__c);                                   \
> >                 __ret = 1;                                              \
> > 
> 
> Yep, making it dependent on CONFIG_DEBUG_BUGVERBOSE makes sense. 
> Andrew, will you take such patch? (when I also fix the 
> evaluating-twice issue).

i'll probably ack such a patch, it can be useful even when the line 
number is unique: if someone reports a WARN_ON() from an old kernel i 
dont have to dig up the exact source but can see it right from the 
condition what happened. Useful redundancy in bug output can be quite 
useful at times. Please post it and we'll see whether it's acceptable.

	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/