<br><br><div class="gmail_quote">On Fri, Apr 2, 2010 at 4:19 PM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">



  

<div bgcolor="#ffffff" text="#000000"><div class="im">
On 4/2/10 7:55 PM, Myles Watson wrote:<br>
<blockquote type="cite">loglevel.diff:<br>
Remove BIOS_EMERG, BIOS_ALERT, BIOS_CRIT, BIOS_NOTICE, BIOS_NEVER<br>
</blockquote></div>
I am not convinced that's a good idea. But let's hear what other people
say...</div></blockquote><div>They were used so infrequently that they were just noise to me.<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div bgcolor="#ffffff" text="#000000"><div class="im"><br>
<blockquote type="cite">I didn't change their numerical values.  They were rarely
used.  <br>
BIOS_EMERG, BIOS_ALERT, BIOS_CRIT -> BIOS_ERR<br>
BIOS_NOTICE -> BIOS_INFO<br>
</blockquote></div>
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.<br><div class="im"></div></div></blockquote><div>I think it preserves the meaning.  Emergency, alert, and critical all seem like errors.  The difference between notice and info seems like splitting hairs.<br>
 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div bgcolor="#ffffff" text="#000000"><div class="im">
<br>
<blockquote type="cite">BIOS_NEVER -> BIOS_SPEW+1<br>
</blockquote></div>
The whole intent was to make clear to the reader that this is something
that's never seen. ;-)<br>
That doesn't become clear from BIOS_SPEW+1 in my opinion.</div></blockquote><div>NEVER was only used once or twice in the code.  If it's something that will be used more often, then I can see your point.  Why don't we just comment out messages that will never be seen?<br>
 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div bgcolor="#ffffff" text="#000000"><div class="im"><br>
<blockquote type="cite">loglevel2.diff:<br>
BIOS_ERR -> ERROR<br>
BIOS_WARNING -> WARNING<br>
BIOS_INFO -> INFO<br>
BIOS_DEBUG -> DEBUG<br>
BIOS_SPEW -> SPEW<br>
For 80 character line aficionados, this puts most of the lines back,
since printk_debug(...) is about the same length as printk(DEBUG, ...)<br>
</blockquote></div>
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<small>.</small></div></blockquote><div>I think printk works well for the prefix, and I like brevity when it doesn't interfere with the meaning.  I think we talked about other prefixes some time in the past and couldn't get agreement.<br>
 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div>I like the idea of not abbreviating ERROR to ERR as the only exception
to the rule.</div></blockquote><div>Good.  I've mis-typed that a lot of times.  I like uniformity.<br><br>Thanks,<br>Myles </div></div><br>