[CRIU] [PATCH 14/14] zdtm: Don't wait too long for restore

Andrey Wagin avagin at gmail.com
Fri Dec 21 10:01:54 EST 2012


2012/12/21 Cyrill Gorcunov <gorcunov at openvz.org>:
> On Fri, Dec 21, 2012 at 06:20:53PM +0400, Andrew Vagin wrote:
>> On Thu, Dec 20, 2012 at 03:55:35PM +0400, Cyrill Gorcunov wrote:
>> >
>> > I've been noticed that we wait too much on restore if the test
>> > is failed. Shrink the time 5 times down.
>>
>> but you shink the time more than 5 times down...
>>
>> sleep 0.1
>> sleep 0.2
>> sleep 0.3
>> ....
>> sleep 0.9
>> sleep 0.9
>> sleep 0.9
>> sleep 0.9
>> sleep 0.9
>
> Dude, see the code :)
>
> - for i in `seq 50`; do
> + for i in `seq 10`; do
>
> 50 -> 10 transition, it's 5 times down, isn't? ;)

Dude, you said about time...

A correct comment should sounds like this:
Shrink the number of iterations 5 times down.

I acked this patch, so we can stop this discussion.
>
>         Cyrill
> _______________________________________________
> CRIU mailing list
> CRIU at openvz.org
> http://lists.openvz.org/mailman/listinfo/criu


More information about the CRIU mailing list