<div dir="ltr"><div class="gmail_default" style="font-size:small">Richard Palethorpe <span dir="ltr"><<a href="mailto:rpalethorpe@suse.de" target="_blank">rpalethorpe@suse.de</a>></span> wrote:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
><br>
> Here we start to get the whole remain time before reaching timeout<br>
> aborting, and base on that to comparing in tst_fzsync_run_a() to guarantee<br>
> test exit when execute time exceed. That's fine, but it is too early to get<br>
> the time here I guess, because samples collection will also cost some of<br>
> the limitation_time(~30 seconds), and we don't know what proportion the<br>
> sampling occupied in exec_time_p, if it's too much, then there only has<br>
> very little time on race execution.<br>
<br>
</div></div>The race can still happen during the sampling period (although for some<br>
tests it is unlikely).<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Sounds right.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
><br>
> To avoid this, maybe we should make the limitation time all spend on race<br>
> execution but any on samplings.<br>
> So, I suggest to get pair->exec_time_start value after samples data<br>
> collection in the end of tst_fzsync_pair_update() function.<br>
<br>
</span>I don't like the sampling time being bounding only by the overall test<br>
timeout. Users really hate test broken messages. Possibly if the test<br>
timeouts before the sampling period has finished, then we could return<br>
TWARN or TCONF saying that the test did not have enough time to execute?<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Sure, that makes users know when timeout occurred, in sampling period or after, at least good for debugging work sometimes.</div></div><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div>
</div></div>