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

Stefan Johansson stefan.johansson at oracle.com
Wed Apr 18 11:55:12 UTC 2018


Thanks for the review.

I will re-run some testing and push this tomorrow.

Thanks,
Stefan

On 2018-04-18 00:27, sangheon.kim wrote:
> 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