[coreboot] [PATCH] continue cleaning up log levels

Stefan Reinauer stepan at coresystems.de
Sat Apr 3 00:19:48 CEST 2010


On 4/2/10 7:55 PM, Myles Watson wrote:
> loglevel.diff:
> Remove BIOS_EMERG, BIOS_ALERT, BIOS_CRIT, BIOS_NOTICE, BIOS_NEVER
I am not convinced that's a good idea. But let's hear what other people
say...
> I didn't change their numerical values.  They were rarely used. 
> BIOS_EMERG, BIOS_ALERT, BIOS_CRIT -> BIOS_ERR
> BIOS_NOTICE -> BIOS_INFO
I don't think we want to just remove log levels like that. Let's rather
fix the messages and choose the right log levels for them.

> BIOS_NEVER -> BIOS_SPEW+1
The whole intent was to make clear to the reader that this is something
that's never seen. ;-)
That doesn't become clear from BIOS_SPEW+1 in my opinion.


> loglevel2.diff:
> BIOS_ERR -> ERROR
> BIOS_WARNING -> WARNING
> BIOS_INFO -> INFO
> BIOS_DEBUG -> DEBUG
> BIOS_SPEW -> SPEW
> For 80 character line aficionados, this puts most of the lines back,
> since printk_debug(...) is about the same length as printk(DEBUG, ...)
I kind of like the fact that they are prefixed, while I see the point
that BIOS_ is a bad prefix. What about LOG_? Because, that's what it is,
log levels.
I like the idea of not abbreviating ERROR to ERR as the only exception
to the rule.

Stefan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100403/de3f8040/attachment.html>


More information about the coreboot mailing list