RFR: 8200228: Change default value of HeapSizePerGCThread

jesper.wilhelmsson at oracle.com jesper.wilhelmsson at oracle.com
Tue Apr 10 13:15:56 UTC 2018


Looks good to me.
/Jesper

> On 10 Apr 2018, at 13:43, Stefan Johansson <stefan.johansson at oracle.com> 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
>>> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Message signed with OpenPGP
URL: <http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/attachments/20180410/0b76e599/signature.asc>


More information about the hotspot-gc-dev mailing list