RFR: 8196071: Change G1 Full GC heap and thread sizing ergonomics

sangheon.kim sangheon.kim at oracle.com
Tue Apr 17 22:27:54 UTC 2018


Hi Stefan,

Webrev.1 looks good to me.

Thanks,
Sangheon


On 04/05/2018 05:05 AM, Stefan Johansson wrote:
> Change looking for second reviewer.
>
> Cheers,
> Stefan
>
> On 2018-03-26 12:12, Thomas Schatzl wrote:
>> Hi,
>>
>> On Mon, 2018-03-26 at 11:47 +0200, Stefan Johansson wrote:
>>>
>>> On 2018-03-26 10:58, Thomas Schatzl wrote:
>>>> Hi,
>>>>
>>>> On Thu, 2018-03-22 at 16:42 +0100, Stefan Johansson wrote:
>>>>> Hi,
>>>>>
>>>>> Please review or comment on this change to let the G1 Full GC
>>>>> calculate the number of worker threads.
>>>>>
>> [...]
>>>>
>>>>     looks good, although I would prefer to separate the
>>>> HeapSizePerGCWorker change into a separate CR.
>>>
>>> Good point. Created JDK-8200228 for this and just realized I
>>> probably
>>> need a CSR for that as well. Here's a new webrev without the flag-
>>> change:
>>> http://cr.openjdk.java.net/~sjohanss/8196071/01/
>>
>>    still good.
>>
>> Thomas
>>



More information about the hotspot-gc-dev mailing list