<div dir="ltr">My argument is that if nothing at all is shown, its difficult to know if that's because nothing needs to be shown, or because the console is busted.<div><br></div><div>Maybe we break break up the banner, and don't print both romstage and ramstage banners, but just the romstage banner.  I'd say that just printing the coreboot version should be sufficient, especially since the build date is based on the commit id:</div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">So at loglevel 0, we get:</span></div><div><span style="font-size:12.8px">  coreboot-4.5-1079-g613350897d</span><br></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">At loglevel 5 (or whatever) we get the additional text:</span></div><div><span style="font-size:12.8px">   coreboot-4.5-1079-g613350897d Fri Feb 24 14:59:42 UTC 2017 romstage starting...</span><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px"><br></span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 24, 2017 at 4:33 PM, Julius Werner <span dir="ltr"><<a href="mailto:jwerner@chromium.org" target="_blank">jwerner@chromium.org</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">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="m_6093175021208991383HOEnZb"><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="m_6093175021208991383h5">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="m_6093175021208991383h5">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<span class="HOEnZb"><font color="#888888"><br></font></span></div></div><span class="HOEnZb"><font color="#888888"><span>--<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></font></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br></font></span></div><span class="HOEnZb"><font color="#888888">
<br>--<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></font></span></blockquote></div><br></div>
</blockquote></div><br></div>