<div dir="ltr"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> > > I also wonder if it is worth somehow put this minimum-enforce<br>
> > > mechanism inside the framework itself<br>
> > > instead that hardcoding it in this specific test (unless you<br>
> > > already mean to do it this way...<br>
> > > and I misunderstood)<br>
> > <br>
> > Yes, I was thinking about it as well.<br>
> > LTP_TIMEOUT_MUL should be reserved for users, tests should use<br>
> > LTP_TIMEOUT_MUL_MIN,<br>
> > check for LTP_TIMEOUT_MUL being higher than LTP_TIMEOUT_MUL_MIN<br>
> > would be in<br>
> > _tst_setup_timer(). Similar mechanism I introduced in 9d6a960d9<br>
> > (VIRT_PERF_THRESHOLD_MIN).<br>
> <br>
> +1 agree.<br>
<br>
I have a general question. What do we try to get with<br>
<span class="gmail_default" style="font-size:small"></span>LTP_TIMEOUT_MUL_MIN? From my perspective, we try to set a minimum<br>
timeout value. Isn't it the value (struct tst_test*)->timeout ?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Well, the (struct tst_test*)->timeout is the default minimum value to set a timeout, but for some test case(e.g memcg_stress_test.sh), they required time should be higher than the default. So as we discussed in the above mails, we're planning to introduce a new variable <span class="gmail_default"></span>LTP_TIMEOUT_MUL_MIN to set as a new minimum value for test timeout. The operation will be encapsulate in function  _tst_setup_timer().</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>
I'm missing such configuration value for shell. Is there one?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">No, we don't have it so far. </div></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>
Or do we need to increase timeout in smaller steps and that is why we<br>
need this LTP_TIMEOUT_MUL_MIN?<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Hmm, what we want to do is:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">If a testcase needs timeout value is larger than the default (300 sec), we could only define a variable LTP_TIMEOUT_MUL_MIN in the test, then the _tst_setup_timer() will detect if LTP_TIMEOUT_MUL_MIN is valid and reset the minimum time for the test.</div><div class="gmail_default" style="font-size:small"><br></div>@Petr and @Cristian<span class="gmail_default" style="font-size:small">, If I misunderstand anything, please correct me.</span><div><br></div></div>-- <br><div dir="ltr" class="m_2765165892186434401gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>