<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Aug 17, 2018 at 3:00 AM Andrey Vagin &lt;<a href="mailto:avagin@virtuozzo.com">avagin@virtuozzo.com</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Apr 28, 2018 at 12:56:40PM +0300, Cyrill Gorcunov wrote:<br>
&gt; The test case may consume more than 5 seconds depending<br>
&gt; on the node load, for this sake we introduce ZDTM_DELTA<br>
&gt; variable (see ae48b5d676e90e998cf6b12d3a36d5f73c45f5ea)<br>
&gt; but actually there was no way to setup it in our vanilla<br>
&gt; zdtm.py engine earlier. Now we can do that so setup the<br>
&gt; limit up to one minute, thus if it trigger it is definitely<br>
&gt; something goes wrong.<br>
&gt; <br>
&gt; <a href="https://ci.openvz.org/view/CRIU/job/CRIU/job/CRIU-snap/job/criu-dev/3717/consoleFull" rel="noreferrer" target="_blank">https://ci.openvz.org/view/CRIU/job/CRIU/job/CRIU-snap/job/criu-dev/3717/consoleFull</a><br>
&gt; <a href="https://ci.openvz.org/view/CRIU/job/CRIU/job/CRIU-x86_64-pre-dump/job/criu-dev/3724/consoleFull" rel="noreferrer" target="_blank">https://ci.openvz.org/view/CRIU/job/CRIU/job/CRIU-x86_64-pre-dump/job/criu-dev/3724/consoleFull</a><br>
&gt; <br>
&gt; Signed-off-by: Cyrill Gorcunov &lt;<a href="mailto:gorcunov@gmail.com" target="_blank">gorcunov@gmail.com</a>&gt;<br>
&gt; ---<br>
&gt;  test/zdtm/static/poll.desc | 2 +-<br>
&gt;  1 file changed, 1 insertion(+), 1 deletion(-)<br>
&gt; <br>
&gt; diff --git a/test/zdtm/static/poll.desc b/test/zdtm/static/poll.desc<br>
&gt; index 63df42aa670a..f8bce6e5d61b 100644<br>
&gt; --- a/test/zdtm/static/poll.desc<br>
&gt; +++ b/test/zdtm/static/poll.desc<br>
&gt; @@ -1 +1 @@<br>
&gt; -{&#39;flavor&#39;: &#39;h&#39;}<br>
&gt; +{&#39;flavor&#39;: &#39;h&#39;, &#39;env&#39;: &#39;ZDTM_DELTA=60&#39;}<br>
<br>
gettimeofday(&amp;time1, NULL)<br>
poll(ufds, 2, delta * 1000);<br>
gettimeofday(&amp;time2, NULL)<br>
<br>
if ((time2.tv_sec - time1.tv_sec) &gt; delta) {<br>
        fail(&quot;Delta is too big %lu&quot;,<br>
                (unsigned long)(time2.tv_sec - time1.tv_sec));<br>
<br>
<br>
I don&#39;t understand how this patch fixes the problem. It is obvious that<br>
time2 - time1 is greater than delta. Maybe we need to take into account<br>
some inaccuracy?<br></blockquote><div><br></div><div>This is mostly for vz7 instance of testing where we have  time virtualized. I&#39;ll review more detailed tomorrow and reply. Sorry for delay.</div></div></div>