Re: [coreboot] [PATCH] Debugging facility improvements

[Uwe Hermann]

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