linux-kernel - Re: [PATCH 2/2] warn: print the DMI board info name in a WARN/WARN

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 for Android: free password hash cracker in your pocket

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

Message-Id: <20081128115247.bfcb603c.akpm@linux-foundation.org>
Date:	Fri, 28 Nov 2008 11:52:47 -0800
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Ingo Molnar <mingo@...e.hu>
Cc:	Arjan van de Ven <arjan@...ux.intel.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] warn: print the DMI board info name in a
 WARN/WARN_ON

On Fri, 28 Nov 2008 18:00:29 +0100 Ingo Molnar <mingo@...e.hu> wrote:

> 
> * Arjan van de Ven <arjan@...ux.intel.com> wrote:
> 
> > From acdc8771dfaf6e8236b9122364301f65f46d4b59 Mon Sep 17 00:00:00 2001
> > From: Arjan van de Ven <arjan@...ux.intel.com>
> > Date: Fri, 28 Nov 2008 08:32:44 -0800
> > Subject: [PATCH] warn: print the DMI board info name in a WARN/WARN_ON
> > 
> > it's very useful for many low level WARN_ON's to find out which
> > motherboard has the broken BIOS etc... this patch adds a printk
> > to the WARN_ON code for this.
> > On architectures without DMI, gcc should optimize the code out.
> > 
> > Signed-off-by: Arjan van de Ven <arjan@...ux.intel.com>
> > ---
> >  kernel/panic.c |    5 +++++
> >  1 files changed, 5 insertions(+), 0 deletions(-)
> > 
> > diff --git a/kernel/panic.c b/kernel/panic.c
> > index 2b913aa..9c1b7e0 100644
> > --- a/kernel/panic.c
> > +++ b/kernel/panic.c
> > @@ -21,6 +21,7 @@
> >  #include <linux/debug_locks.h>
> >  #include <linux/random.h>
> >  #include <linux/kallsyms.h>
> > +#include <linux/dmi.h>
> >  
> >  int panic_on_oops;
> >  static unsigned long tainted_mask;
> > @@ -328,12 +329,16 @@ void warn_slowpath(const char *file, int line, const char *fmt, ...)
> >  {
> >  	va_list args;
> >  	char function[KSYM_SYMBOL_LEN];
> > +	const char *board;
> >  	unsigned long caller = (unsigned long)__builtin_return_address(0);
> >  	sprint_symbol(function, caller);
> >  
> >  	printk(KERN_WARNING "------------[ cut here ]------------\n");
> >  	printk(KERN_WARNING "WARNING: at %s:%d %s()\n", file,
> >  		line, function);
> > +	board = dmi_get_system_info(DMI_PRODUCT_NAME);
> > +	if (board)
> > +		printk(KERN_WARNING "Hardware name: %s\n", board);
> >  
> >  	if (fmt) {
> >  		va_start(args, fmt);
> 
> nice idea - applied to tip/core/debug.
> 
> On architectures (or systems) where there's no DMI_PRODUCT_NAME string 
> available, we'll print nothing. It's also a simple lockless call so can 
> be called from any context.
> 
> Andrew, any objections?
> 

Nope.  This is the first time we've included dmi.h from such a core
place, but afaict all architectures should have encountered that file
by now.

--
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/