<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 Mon, Sep 19, 2022 at 4:26 PM 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">Hi!<br>
> > tst_pollute_memory() consume time is proportional to the amount of<br>
> > free RAM, it is hard to find one fixed value of max_runtime to fit all test<br>
> > platforms.<br>
> ><br>
> > From my experience, if you limited this test only run with small<br>
> > machine (e.g. RAM <= 32G), that performs well with whatever<br>
> > bare metal or VM, no timeout ever.<br>
> ><br>
> <br>
> Btw, we did that by setting a test filter before LTP running, also we could<br>
> add a field .max_mem_avail to tst_test struct for achieving that, but not<br>
> sure if it's worth doing that at this moment.<br>
<br>
The proper solution will be adding a separate timeouts for setup/cleanup<br>
and limiting the setup runtime to something sensible, but that is<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Separate timeouts for setup/cleanup will break the integrity of</div><div class="gmail_default" style="font-size:small">setting functions, my concern is that if tst_pollute_memory stopped</div><div class="gmail_default" style="font-size:small">in uncompleted, the main test part is meaningless, right?</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Or, I may misunderstand you here.</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">
something for the next development cycle.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">+1</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>