RFR: 8304712: Only pass total number of regions into G1Policy::calc_min_old_cset_length [v2]

Thomas Schatzl tschatzl at openjdk.org
Thu Mar 23 19:13:28 UTC 2023


On Thu, 23 Mar 2023 12:01:57 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Hi all,
>> 
>>   please review this change that refactors `G1Policy::calc_min_old_cset_length` to take only required parameters and also just use `ceil()` for the calculation. It just does not matter to be clever here imo.
>> 
>> Testing: gha
>> 
>> Thanks,
>>   Thomas
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Rename back argument

Thanks for your reviews

-------------

PR Comment: https://git.openjdk.org/jdk/pull/13147#issuecomment-1481753391


More information about the hotspot-gc-dev mailing list