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

[Zoltán Majó]

On 07/21/2016 10:38 AM, Dmitry Fazunenko wrote:
> [...]
>
> On 21.07.2016 11:30, Zoltán Majó wrote:
>>
>> To make sure we don't forget to un-quarantine those test, could you 
>> please file a sub-task for un-quarantining? We could then use the JBS 
>> ID of that sub-task to push the un-quarantining change.
>
> Done: https://bugs.openjdk.java.net/browse/JDK-8161990
>
> Thanks again for the prompt review.

Thank you, Dima!

Best regards,


Zoltan

> Dima
>
>>
>> Thank you!
>>
>> Best regards,
>>
>>
>> Zoltan
>>
>>>
>>> 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
>>>>
>>>
>>
>