RFR: 8200228: Change default value of HeapSizePerGCThread

Aleksey Shipilev shade at redhat.com
Tue Apr 10 13:18:34 UTC 2018


This change is fine with me. Count me as Reviewer.

-Aleksey

On 04/10/2018 01:43 PM, Stefan Johansson wrote:
> CSR is now approved. Do I need a second reviewer for this default value change?
> 
> Stefan
> 
> On 2018-03-29 10:52, Stefan Johansson wrote:
>> I've now created a CSR as well:
>> https://bugs.openjdk.java.net/browse/JDK-8200417
>>
>> Please have a look at it and add yourself as a reviewer if everything looks good.
>>
>> Thanks,
>> Stefan
>>
>> On 2018-03-28 15:52, Stefan Johansson wrote:
>>> Hi,
>>>
>>> Please review this change of the default value for HeapSizePerGCThread.
>>>
>>> Links
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8200228
>>> Webrev: http://cr.openjdk.java.net/~sjohanss/8200228/00/
>>>
>>> Summary
>>> My testing shows that for DaCapo, SpecJVM98 and GCBasher (gc intensive stress test) with small
>>> heaps we get better pause times with the flag set to 32M. There might be other arguments, such as
>>> not using to much resources on small systems that might also be important so please let me know
>>> if you have reasons why the flag should be left unchanged.
>>>
>>> Thanks,
>>> Stefan
>>>
>>



More information about the hotspot-gc-dev mailing list