[coreboot] [PATCH] Debugging facility improvements

Uwe Hermann uwe at hermann-uwe.de
Sun Oct 24 15:29:08 CEST 2010


On Tue, Oct 12, 2010 at 09:46:47AM +0200, Peter Stuge wrote:
> Uwe Hermann wrote:
> > Debugging facility improvements.
> > 
> >  - Hook up malloc() debug code via CONFIG_DEBUG_MALLOC.
> > 
> >  - Get rid of a custom "debug" macro, use printk() as usual.
> > 
> > Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
> > 
> > Index: src/Kconfig
> > ===================================================================
> > --- src/Kconfig	(Revision 5940)
> > +++ src/Kconfig	(Arbeitskopie)
> > @@ -553,6 +553,16 @@
> >  
> >  	  If unsure, say N.
> >  
> > +config DEBUG_MALLOC
> > +	bool "Output verbose malloc debug messages"
> > +	default n
> > +	help
> > +	  This option enables additional malloc related debug messages.
> > +
> > +	  Note: This option will increase the size of the coreboot image.
> > +
> > +	  If unsure, say N.
> 
> Should this also force debug level to SPEW?

No idea, but it's unrelated to this patch. As far as I can see all other
such DEBUG_* mechanisms don't force the debug level to SPEW either, so
if we want to do that (I'm not sure we do) that's material for another
patch.


Uwe.
-- 
http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org




More information about the coreboot mailing list