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 12:01:57 UTC 2023


> 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

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/13147/files
  - new: https://git.openjdk.org/jdk/pull/13147/files/13c093c6..544efb13

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=13147&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=13147&range=00-01

  Stats: 3 lines in 2 files changed: 0 ins; 0 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/13147.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/13147/head:pull/13147

PR: https://git.openjdk.org/jdk/pull/13147


More information about the hotspot-gc-dev mailing list