RFR: 8222182: ZGC: Use SoftMaxHeapSize to guide GC heuristics

Per Liden per.liden at oracle.com
Thu Apr 11 14:12:44 UTC 2019


Hi Thomas,

On 04/11/2019 02:33 PM, Thomas Schatzl wrote:
> Hi Per,
> 
> On Thu, 2019-04-11 at 13:50 +0200, Per Liden wrote:
>> Hi Thomas,
>>
>> Thank for looking at the patch. However, I'm afraid your comments
>> are based on incorrect assumptions, which makes them difficult to
>> address.
>> I'll try to clarify a few key points below. Hope that helps.
> 
>    after looking at your comments, I got the source of my
> misunderstanding: I misread the change in zPageAllocator.cpp:112 to be
> a new version of ZPageAllocator::current_max_capacity(), but it changed
> to ZPageAllocator::soft_max_capacity().
> 
> And the old contents of ZPageAllocator::current_max_capacity() has been
> inlined into ZPageAllocator::max_available().
> 
> Sorry for the fuss. Looks good to me then.

Great! No problem, thanks for reviewing.

> 
> Still I strongly recommend a release note :)

That's a good idea and I will add one.

cheers,
Per

> 
> Thanks,
>    Thomas
> 
> 



More information about the hotspot-gc-dev mailing list