RFR (XXS): 8164133: Tests gc/arguments/TestAlignmentToUseLargePages.java and gc/cms/TestBubbleUpRef.java use too small heap

[Jon Masamitsu]

On 08/22/2016 08:55 AM, Dmitry Fazunenko wrote:
> Jon,
>
> The problem could be reproduced by the following command:
>
> #> java -Xmx3m -XX:+UseConcMarkSweepGC -XX:+AggressiveOpts -version
> Error occurred during initialization of VM
> GC triggered before VM initialization completed. Try increasing 
> NewSize, current value 640K.

I didn't notice that "-Xmx3m" was on the command line for the original 
failure.
Without -Xmx3m I expected that the default heap was large and wondered why
NewSize was so small.

Jon

>
> increasing Xmx from 3m to 4m helps:
>
> #> java -Xmx4m -XX:+UseConcMarkSweepGC -XX:+AggressiveOpts -version
> java version "9-ea"
> Java(TM) SE Runtime Environment (build 9-ea+132)
> Java HotSpot(TM) 64-Bit Server VM (build 9-ea+132, mixed mode)
>
> I don't see anything suspicious in the GC log (attached).
>
> There is no requirement to minimal heap required for Hotspot to work 
> normally, so I think the tests should not start VM with so small heaps.
>
>
> Thanks,
> Dima
>
>
>
>
> On 19.08.2016 21:02, Jon Masamitsu wrote:
>> Dima,
>>
>> The error message I see in the logs is
>>
>> GC triggered before VM initialization completed. Try increasing 
>> NewSize, current value 1280K.
>>
>> Do you know why the NewSize is so small?  I ask because I'm
>> not sure setting a larger heap size will always fix this failure.
>>
>> If the test is run (with the parameters that fail) and 
>> -Xlog:gc*=trace, you might
>> see why the NewSize is so small (if you don't know now).
>>
>> Jon
>>
>>
>> On 8/19/2016 9:02 AM, Dmitry Fazunenko wrote:
>>> Hello,
>>>
>>> May I have to get a couple of reviews for a trivial fix:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8164133
>>> http://cr.openjdk.java.net/~dfazunen/8164133/webrev.01/
>>>
>>> Two tests set very small maximum heap size, which may cause:
>>>   starts VM with too small heap size, which might cause
>>>   Error occurred during initialization of VM
>>> None of those tests does really require such a small heap, so just 
>>> increasing should help.
>>>
>>> Thanks,
>>> Dima
>>>
>>>
>>>
>>
>