RFR(trivial): 8210102: CDS tests timed out

[Calvin Cheung]

Thanks, Ioi!

On 11/29/18, 5:12 PM, Ioi Lam wrote:
> 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
>