RFR: 8168396: Unexpected OOME in GcCauseTest02 and GcTest02

Dmitry Fazunenko dmitry.fazunenko at oracle.com
Thu Oct 27 16:05:54 UTC 2016


Michail,

Thanks for addressing my comments. New version looks good to me.

--Dima

On 27/10/2016 16:23, Michail Chernov wrote:
> Hi,
>
> I'd like to update the webrev after offline discussion with Dmitry 
> Fazunenko. Classes structure was simplified, removed unnecessary 
> interface.
>
> Tests GcCauseTest02 and GcTest02 were updated - old version can throw 
> OOME. The reason of OOME is that tests do not consider available free 
> heap and try to fill 70% of all heap. This change makes heap size 
> fixed ( InitialHeapSize == MaxHeapSize ). New version tries to make 
> heap usage more than 70% and invokes System.gc() to update performance 
> counters/pools statistics.
>
> http://cr.openjdk.java.net/~mchernov/8168396/webrev.01/
>
> Thanks,
> Michail
>
> On 26/10/2016 16:02, Michail Chernov wrote:
>> Hi,
>>
>> Could you please review this change?
>>
>> http://cr.openjdk.java.net/~mchernov/8168396/webrev.00/
>> https://bugs.openjdk.java.net/browse/JDK-8168396
>>
>> Initial heap size and max heap size were made equal to guarantee 
>> expected memory allocation. Previous version of test can throw OOME 
>> because does not consider existing free heap size.
>>
>> Thanks,
>> Michail
>>
>




More information about the hotspot-gc-dev mailing list