RFR 9 test-only RFR 8177328 : java/lang/ClassLoader/securityManager/ClassLoaderTest.java times out with -Xcomp

[joe darcy]

I understand the interest in having test pass reliably, but I don't 
think giving the test very large timeouts is the preferred way of 
accomplishing that.

For all configurations, the test can now run for up to 20 minutes, up 
from 4 minutes. We want to run the entire test suite, thousands of 
tests, in about 20 minutes. The the timeout factor used for Xcomp run, 
the test would probably now be able to run for over an hour before 
timing out.

I suggest making the test run faster, or seeing if there has been a 
regressions in Xcomp to make test perform more poorly there.

Thanks,

-Joe


On 4/27/2017 12:08 PM, Brent Christian wrote:
> Hi,
>
> This test times out under our automated testing configurations that 
> include -Xcomp.
>
> Please review my change to increase the timeout for this test.  It is 
> sufficient for the test configurations in question to pass on two 
> different local machines (Mac & Linux).
>
> diff -r 7c04ab31b4d6 
> test/java/lang/ClassLoader/securityManager/ClassLoaderTest.java
> --- a/test/java/lang/ClassLoader/securityManager/ClassLoaderTest.java 
> Wed Apr 26 09:37:23 2017 -0700
> +++ b/test/java/lang/ClassLoader/securityManager/ClassLoaderTest.java 
> Thu Apr 27 12:03:33 2017 -0700
> @@ -29,7 +29,7 @@
>   * @library /lib/testlibrary
>   * @modules java.base/jdk.internal.module
>   * @build JarUtils CompilerUtils
> - * @run main/timeout=240 ClassLoaderTest
> + * @run main/timeout=1200 ClassLoaderTest
>   */
>
> Thanks,
> -Brent