<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@redhat.com</a>> wrote:<br></div><div class="gmail_default" style="font-size:small"><br></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Some systems (specially with combination of -debug kernel<br>
with KASAN enabled) have trouble completing this test<br>
in specified timeout.<br>
<br>
Lowering number of iterations would make the test condition<br>
less accurate as it's based on global counter.<br>
<br>
Instead, calculate the rate of iterations system can do in<br>
first 5 seconds and used that to decide whether to continue<br>
to run the test. If the rate is too slow, TCONF after 5<br>
seconds.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Generally, this method looks good, but maybe better to limit this</div><div class="gmail_default" style="font-size:small">check_progress() only perform on -debug kernel?  Otherwise,</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>><br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">======= FYI ==========</div><div class="gmail_default" style="font-size:small">I'm seeking a fair way to make a global evaluation of the test</div><div class="gmail_default" style="font-size:small">system to reset timeout dynamically for the whole LTP.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">My original design thoughts:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">  Create the numbers of threads equal to CPUs and bind them to</div><div class="gmail_default" style="font-size:small">  the corresponding cpu for running. Use mutex lock to sync up</div><div class="gmail_default" style="font-size:small">  each thread launch at the same time to collect the basic data</div><div class="gmail_default" style="font-size:small">  for their CPU. Then we can compare the CPU state under the idle or</div><div class="gmail_default" style="font-size:small">  busy time to get a relatively stationary _value_ to measure the system</div><div class="gmail_default" style="font-size:small">  performance.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small"><div class="gmail_default">But so far the test method is not stable&reliable as expected.</div><div class="gmail_default"><br></div><div class="gmail_default"></div></div><div class="gmail_default" style="font-size:small">  // do float computing + dirty 10*pagesz memory  in a limited times<br></div><div class="gmail_default" style="font-size:small">  one_unit_of_operation();</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">  // count the CPU looping numbers with (type = idel, calcu)</div><div class="gmail_default" style="font-size:small">  // and call one_unit_opertaion() in 1 sec</div><div class="gmail_default" style="font-size:small">  cpu_1sec_looping(int type);<br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">  idlespeed_loops = cpu_1sec_looping(idel);</div><div class="gmail_default" style="font-size:small">  calculate_loops = cpu_1sec_looping(calcu);</div><div class="gmail_default" style="font-size:small">  ...</div><div class="gmail_default" style="font-size:small">  // count the _value_ from all CPU average loops</div><div class="gmail_default" style="font-size:small">  ratio = calculate_avg / idealspeed_avg;</div></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>