RFR: 8171045: gc/stress/TestStressG1Humongous.java fails to allocate the heap

Michail Chernov michail.chernov at oracle.com
Tue Dec 20 17:04:52 UTC 2016


Kirill,

Thank you for reviewing this. I filed RFE for this improvement - 
https://bugs.openjdk.java.net/browse/JDK-8171512


Thanks,
Michail

On 20/12/2016 17:10, Kirill Zhaldybin wrote:
> Michail,
>
> Looks good to me.
>
> Bit of nitpicking - it could look nicer if you sort @runs basing on 
> G1HeapRegionSize (from 1M to 32M or vice versa) but it's up to you.
>
> Regards, Kirill
>
> On 20.12.2016 16:59, Michail Chernov wrote:
>> Hi,
>>
>> Could I have a reviews for this small change for one test, please?
>>
>> http://cr.openjdk.java.net/~mchernov/8171045/webrev.00/
>> https://bugs.openjdk.java.net/browse/JDK-8171045
>>
>> Sometimes test failed with error:
>>
>> Error occurred during initialization of VM
>> Could not reserve enough space for 1048576KB object heap
>>
>> This happens on 32 bit systems. Now the test limits heap size to 512 M.
>>
>> Testing via RBT was done.
>>
>> Thanks,
>> Michail
>
>




More information about the hotspot-gc-dev mailing list