linux-kernel - Re: [PATCH] more helpful WARN_ON and BUG

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: <20061021024105.GA17706@redhat.com>
Date:	Fri, 20 Oct 2006 22:41:05 -0400
From:	Dave Jones <davej@...hat.com>
To:	Adrian Bunk <bunk@...sta.de>
Cc:	Eric Sandeen <sandeen@...hat.com>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] more helpful WARN_ON and BUG_ON messages

On Sat, Oct 21, 2006 at 12:07:17AM +0200, Adrian Bunk wrote:

 > > Most debugging code makes the kernel bigger, slower... and easier to
 > > debug, no?
 > > 
 > > It's not a question of not being -able- to locate sources; it's a
 > > question of being able to look at a bug report and triage it quickly
 > > without digging around to find the kernel du jour that produced it.  *shrug*
 > 
 > It's not that BUGs were that frequent.

You're not trying hard enough ;) 

 > And with your suggestion "I suppose this could be put under CONFIG_DEBUG", 
 > it would anyway be turned off by nearly everyone.

For better bug reports, 16K is peanuts. We had exactly the same straw-man
come up when kksymoops was first proposed. And now, most people don't run
without it.

I've seen numerous cases where reporters have hand transcribed BUG() reports,
and got the line numbers wrong because they misremembered a 4 digit number.
Words are inherently easier to remember, and even if typoed, usually there's
enough context to figure out what the problem was without another round-trip
to the bug reporter.

If this were optional, I don't see how anyone can argue against it,
and that should be a trivial improvement to Eric's existing patch.

	Dave

-- 
http://www.codemonkey.org.uk
-
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/