RFR(xs): 8161552: Test issue: VM init failed: GC triggered before VM initialization completed. Try increasing NewSize, current value 768K.

[Dmitry Fazunenko]

Hi Zoltan,

Thanks for reviewing!

The tests are currently quarantined in jdk9/dev, but not in jdk9/hs yet.
To test them properly I prefer to integrate through the jdk9/hs repository.
I will either wait for sync or un-quarantine as a separate change.

Thanks,
Dima

On 21.07.2016 10:49, Zoltán Majó wrote:
> Hi Dima,
>
>
> thank you for taking care of this issue, your changes look generally 
> good to me. But I would be interested in whether you plan to 
> un-quarantine the two tests that originally failed:
>
> gc/arguments/TestParallelHeapSizeFlags.java
> gc/TestSmallHeap.java
>
> If yes, can you maybe do that with the current changeset? If not, can 
> you please explain why?
>
> Thank you!
>
> Best regards,
>
>
> Zoltan
>
> On 07/20/2016 07:01 PM, Dmitry Fazunenko wrote:
>> Hello,
>>
>> I'm looking for a couple of reviewers for a simple fix of two tests.
>> The tests check some basic assertions when VM is started with max 
>> heap size set to some small values.
>> The tests work almost everywhere, but there are cases when  with 
>> -Xmx2M GC triggered before VM initialization completed.
>> 2M is valid but extremely small max heap size. Cases of usages such 
>> small heap are rather theoretical than practical.
>>
>> To fix the problem I made the tests weaker, the cases setting max 
>> heap to 2M are removed.
>>
>> http://cr.openjdk.java.net/~dfazunen/8161552/webrev.00/
>> https://bugs.openjdk.java.net/browse/JDK-8161552
>>
>> Testing: locally + RBT
>>
>> Thanks,
>> Dima
>