This is my second attempt at fixing this timeout by taking the jtreg timeout 
factor into account in the tests. The first fix [1] looked at the wrong 
environment variable, and also would have caused the test to run unnecessarily 
slow since it set sleep_seconds to a higher value instead of changing the 
timeout.

In this version I have tried to fix these problems. I now look at the env 
variable TESTTIMEOUTFACTOR which will contain the jtreg timeout factor in 
floating point notation. Since it is easier to work with integers in shell 
scripts, I truncate this value using and awk expression. I then use this value 
to set up time limits in the two places where we have them. To simplify the 
code in the cmd() function, I no longer print out stack traces after half the 
timeLimit, only when the limit has expired. I think this is reasonable.

bug: https://bugs.openjdk.java.net/browse/JDK-8157555 
<https://bugs.openjdk.java.net/browse/JDK-8157555>webrev: 
http://cr.openjdk.java.net/~sla/8157555/webrev.00/ 
<http://cr.openjdk.java.net/~sla/8157555/webrev.00/>

Thanks,
/Staffan

[1] http://hg.openjdk.java.net/jdk9/hs-rt/jdk/rev/5a553039e9fc

Reply via email to