RFR(trivial): 8210102: CDS tests timed out

[Ioi Lam]

Hi Calvin,

The fixes looks good and trivial.

Thanks
- Ioi

On 11/29/18 5:03 PM, Calvin Cheung wrote:
> bug: https://bugs.openjdk.java.net/browse/JDK-8210102
>
> Among the three tests mentioned in the bug report, only the 
> DifferentHeapSizes test timeout could be reproduced on solaris-sparcv9 
> with the tier6 vm options (-XX:MaxRAMPercentage=12 -Xcomp 
> -XX:+CreateCoredumpOnCrash -ea -esa -XX:CompileThreshold=100 
> -XX:+UnlockExperimentalVMOptions -server -XX:-TieredCompilation).
> Two timeouts out of a total of 40 runs.
>
> timed out (timeout set to 960000ms, elapsed time including timeout 
> handling was 1199805ms)
> timed out (timeout set to 960000ms, elapsed time including timeout 
> handling was 1175265ms)
>
> Increasing the timeout from 120s (default) to 160s seems to be 
> sufficient. No timeout was observed out of a total of 80 runs. With 
> the current timeout factor setting of 8, a 160s timeout allows a total 
> time of 160x8 = 1280s.
>
> Here's the patch:
>
> diff --git 
> a/test/hotspot/jtreg/runtime/appcds/cacheObject/DifferentHeapSizes.java 
> b/test/hotspot/jtreg/runtime/appcds/cacheObject/DifferentHeapSizes.java
> --- 
> a/test/hotspot/jtreg/runtime/appcds/cacheObject/DifferentHeapSizes.java
> +++ 
> b/test/hotspot/jtreg/runtime/appcds/cacheObject/DifferentHeapSizes.java
> @@ -31,7 +31,7 @@
>   * @compile ../test-classes/Hello.java
>   * @build sun.hotspot.WhiteBox
>   * @run driver ClassFileInstaller sun.hotspot.WhiteBox
> - * @run main/othervm -XX:+UnlockDiagnosticVMOptions -XX:+WhiteBoxAPI 
> -Xbootclasspath/a:. DifferentHeapSizes
> + * @run main/othervm/timeout=160 -XX:+UnlockDiagnosticVMOptions 
> -XX:+WhiteBoxAPI -Xbootclasspath/a:. DifferentHeapSizes
>   */
>
> thanks,
> Calvin