<div class="gmail_quote">On Mon, Nov 1, 2010 at 8:04 PM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div><div></div><div class="h5">On 02.11.2010 02:20, David Hendricks wrote:<br>
> New version -- fixes the number of regions tested (i had reduced them to get<br>
> a patch prepped in a hurry. d'oh!)<br>
><br>
> On Mon, Nov 1, 2010 at 6:10 PM, David Hendricks <<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>> wrote:<br>
><br>
><br>
>> We were a bit too aggressive in trimming superfluous print messages earlier<br>
>> on. This patch adds a line of output for each iteration of the while loops<br>
>> in the test. The intention is to give the user something useful to indicate<br>
>> test progress without just printing a ton of debug spew. Maybe a future<br>
>> patch can redirect more verbose info to a log file.<br>
>><br>
>> The output looks like this::<br>
>> localhost ~ # FLASHROM="./flashrom" sh flashrom_partial_write_test.sh<br>
>> testing flashrom binary: ./flashrom<br>
>> Running test in /tmp/tmp.4xPejwaADU<br>
>> ffh pattern written in ff_4k.bin<br>
>> 00h pattern written in 00_4k.bin<br>
>> Reading BIOS image<br>
>> Original image saved as bios.bin<br>
>> aligned region 0 test: passed<br>
>> ...<br>
>> aligned region 15 test: passed<br>
>> unaligned region 0 test: passed<br>
>> ...<br>
>> unaligned region 15 test: passed<br>
>> Result: PASSED<br>
>> restoring original bios image using system's flashrom<br>
>> test files remain in /tmp/tmp.4xPejwaADU<br>
>><br>
>> Signed-off-by: David Hendricks <<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>><br>
>><br>
<br>
</div></div>Looks good and improves the output.<br>
I removed the /dev/null redirect of the final flashrom run since we<br>
really want to know if the fail flashrom run had any issues.<br></blockquote><div><br></div><div>Sounds good. Thanks!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">


<br>
Acked-by: Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>><br>
and committed in r1222.<br>
<br>
Regards,<br>
Carl-Daniel<br>
<font color="#888888"><br>
--<br>
<a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br>
<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>