<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 26, 2021 at 12:02 AM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Rarely there is a need to set the test runtime dynamically, the only<br>
tests in LTP that does this are the timer tests that can get two<br>
parameters, number of iterations and sleep time, and the test runtime is<br>
close to the multiplication of these two.<br>
<br>
It's still cleaner to set the runtime and let the test library figure<br>
out the timeout in this case.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">If so, should we consider to hinden the .timeout in struct tst_test </div><div class="gmail_default" style="font-size:small">to prevent users from changing it?</div><br></div><div><div class="gmail_default" style="font-size:small">IIRC, we currently have ".timeout == -1" to disable test timed </div><div class="gmail_default" style="font-size:small">out in unsure situation, e.g some OOM tests. But in this patch,</div><div class="gmail_default" style="font-size:small">I saw you remove that, but not handle it in tst_set_runtime.</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Also when no parameters are passed to these tests the runtime is a sum<br>
of multiplications from the tst_timer_test.c source so we define a<br>
constant in the tst_timer_test.h header and set the max_runtime in the<br>
testcases accordingly. With this we get correct estimate for the test<br>
runtime and tighter, but still forgiving enough, timeout as well.<br></blockquote><div><br></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>