Looks good to me. --alex
On 19.05.2016 15:11, Sergey Bylokhov wrote:
Hello, Audio Guru. Please review the small fix for unstable test reported by mach5 in jdk9. The test has two issues: - It uses System.currentTimeMillis() which can be affected by the system time change. Replaced by System.nanoTime(). - It maintains its own timeout(5 seconds) to catch the hang of the test. It was found that on some overloaded system this period is not enough(on the tested system was necessary to increase it to 30-40 seconds or it will fail a few times per 1000 executions). I removed this internal timeout from the test(it is provided as the warning now), and relies on jtreg default timeout. It has a benefit that when the test hangs the jtreg will provide a jstack output which will be useful to understand where is the problem. Bug: https://bugs.openjdk.java.net/browse/JDK-8148915 Webrev can be found at: http://cr.openjdk.java.net/~serb/8148915/webrev.00/
