<div dir="ltr">I think "only print problems / exceptional conditions" is a reasonable use case. All UART output wastes time.<div><br></div><div>Is there a reason you want to see the banner but don't want to run at NOTICE? Maybe the better answer would be to add a new log level (e.g. LOG_IMPORTANT) between NOTICE and WARNING (or maybe between WARNING and ERROR)? Do you think there's a case where you would not want to see actual errors, but still want to see the banner? (Of course it's possible that on some platforms even LOG_ERROR spews uninteresting spam right now, but we should probably consider that a bug with the code that does so instead.)</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 24, 2017 at 2:05 PM, Martin Roth <span dir="ltr"><<a href="mailto:gaumless@gmail.com" target="_blank">gaumless@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Paul,<div>  I agree that it seems reasonable to always print the banner to the console. My recommendation is that we don't even add it as an option.  If there's a console set up, write the banner to it.</div><div><br></div><div>As I recall from Sage's version of coreboot, they made this same change, and printed it at level 0.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Martin</div><div><br></div><div><br></div></font></span></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Fri, Feb 24, 2017 at 2:52 PM, Paul Menzel via coreboot <span dir="ltr"><<a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">Dear coreboot folks,<br>
<br>
<br>
Playing a little with the log levels and QEMU, it turns out that there<br>
are no coreboot messages below the log level *NOTICE*.<br>
<br>
This is expected, but like with SeaBIOS, it’d be a good idea in my<br>
opinion to at least output the “banner” of romstage and ramstage.<br>
<br>
```<br>
coreboot-4.5-1079-g613350897d Fri Feb 24 14:59:42 UTC 2017 romstage starting...<br>
<br>
coreboot-4.5-1079-g613350897d Fri Feb 24 14:59:42 UTC 2017 ramstage starting...<br>
```<br>
<br>
This doesn’t map with the current log level description. Here are the<br>
log levels with their help text from `src/console/Kconfig`.<br>
<br>
> config DEFAULT_CONSOLE_LOGLEVEL_8<br>
>   bool "8: SPEW"<br>
>   help<br>
>     Way too many details.<br>
> config DEFAULT_CONSOLE_LOGLEVEL_7<br>
>   bool "7: DEBUG"<br>
>   help<br>
>     Debug-level messages.<br>
> config DEFAULT_CONSOLE_LOGLEVEL_6<br>
>   bool "6: INFO"<br>
>   help<br>
>     Informational messages.<br>
> config DEFAULT_CONSOLE_LOGLEVEL_5<br>
>   bool "5: NOTICE"<br>
>   help<br>
>     Normal but significant conditions.<br>
> config DEFAULT_CONSOLE_LOGLEVEL_4<br>
>   bool "4: WARNING"<br>
>   help<br>
>     Warning conditions.<br>
> config DEFAULT_CONSOLE_LOGLEVEL_3<br>
>   bool "3: ERR"<br>
>   help<br>
>     Error conditions.<br>
> config DEFAULT_CONSOLE_LOGLEVEL_2<br>
>   bool "2: CRIT"<br>
>   help<br>
>     Critical conditions.<br>
> config DEFAULT_CONSOLE_LOGLEVEL_1<br>
>   bool "1: ALERT"<br>
>   help<br>
>     Action must be taken immediately.<br>
> config DEFAULT_CONSOLE_LOGLEVEL_0<br>
>   bool "0: EMERG"<br>
>   help<br>
>     System is unusable.<br>
<br>
In my opinion, it’d be a good idea to make an exception for the banner.<br>
<br>
What do you think? Another solution would be, to make that a separate<br>
Kconfig option like `CONSOLE_ALWAYS_PRINT_BANNER`, but I’d like to not<br>
add other configuration options, if it’s not needed.<br>
<br>
<br>
Thanks,<br>
<br>
Paul<br></div></div><span class="">--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a><br>
<a href="https://www.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" target="_blank">https://www.coreboot.org/mailm<wbr>an/listinfo/coreboot</a><br></span></blockquote></div><br></div>
<br>--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="https://www.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" target="_blank">https://www.coreboot.org/<wbr>mailman/listinfo/coreboot</a><br></blockquote></div><br></div>