New version -- fixes the number of regions tested (i had reduced them to get a patch prepped in a hurry. d'oh!)<div><br></div><div><div class="gmail_quote">On Mon, Nov 1, 2010 at 6:10 PM, David Hendricks <span dir="ltr"><<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">We were a bit too aggressive in trimming superfluous print messages earlier on. This patch adds a line of output for each iteration of the while loops in the test. The intention is to give the user something useful to indicate test progress without just printing a ton of debug spew. Maybe a future patch can redirect more verbose info to a log file.<div>


<br></div><div>The output looks like this::<div><div><div><font face="'courier new', monospace"><div>localhost ~ # FLASHROM="./flashrom" sh flashrom_partial_write_test.sh </div><div>
testing flashrom binary: ./flashrom</div><div>Running test in /tmp/tmp.4xPejwaADU</div><div>ffh pattern written in ff_4k.bin</div><div>00h pattern written in 00_4k.bin</div><div>Reading BIOS image</div><div>Original image saved as bios.bin</div>


<div>aligned region 0 test: passed</div><div>...</div><div>aligned region 15 test: passed</div>
<div>unaligned region 0 test: passed</div><div>...</div><div>unaligned region 15 test: passed</div><div>Result: PASSED</div><div>restoring original bios image using system's flashrom</div><div>test files remain in /tmp/tmp.4xPejwaADU</div>


<div><br></div></font></div><div>Signed-off-by: David Hendricks <<a href="mailto:dhendrix@google.com" target="_blank">dhendrix@google.com</a>></div><div><br></div>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>



</div></div></div>
</blockquote></div><br><br clear="all"><br>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>
</div>