On Wed, Nov 12, 2008 at 10:08 PM, Mart Raudsepp <span dir="ltr"><<a href="mailto:mart.raudsepp@artecdesign.ee">mart.raudsepp@artecdesign.ee</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On N, 2008-11-13 at 03:14 +0100, Carl-Daniel Hailfinger wrote:<br>
> On 13.11.2008 02:43, Peter Stuge wrote:<br>
> > Carl-Daniel Hailfinger wrote:<br>
> ><br>
> >>>> Be less noisy in ram_check in non-debug loglevels<br>
> >>>><br>
> >> I'm not so totally sure we really want this change. I won't veto it,<br>
> >> but I think RAM test errors should always be shown with full<br>
> >> verbosity.<br>
> >><br>
> ><br>
> > Because the result is really meaningless in the general case I like<br>
> > the patch.<br>
> ><br>
><br>
> You managed to snip my suggested solution when you quoted my mail. Here<br>
> it is again for your convenience:<br>
><br>
> >> If it is really needed, you could add another parameter to<br>
> >> ram_check which specifies the log level.<br>
<br>
<br>
</div>I'd actually go as far as rip out the last printk too, or rather always<br>
have it BIOS_DEBUG, not BIOS_ERR when it fails. Rationale is the<br>
following:<br>
<br>
<br>
ram_check has three viable use cases that I can think of:<br>
<br>
a) During development as a local addition of a call to it while doing<br>
memory timing fixing and work<br>
b) To do automatic memory sizing a la DBE61<br>
c) To react somehow to memory failure, perhaps disabling one DIMM out of<br>
multiple or loading a payload that runs in CAR that instructs the user<br>
to fix his RAM or whatever.<br>
<br>
b) and c) means its used in production code that shouldn't be spitting<br>
all this failure spam into serial, but instead the return value of<br>
ram_check should be checked by the caller and reacted upon (goes<br>
together with the warn_unused suggestion)</blockquote><div><br>If you're running it in production code to detect the ram size, do you really want to send an error message after you find the (expected) end of ram? In other words, do you want to have an error message show up on every boot displaying an error message about a ram failure, and the address of that failure, that an end user (familiar only with typical BIOSes) might believe to be a bad thing?<br>
<br>-Corey<br></div></div>