RFR: 8331562: G1: Remove API to force allocation of new regions [v2]
Thomas Schatzl
tschatzl at openjdk.org
Fri May 3 12:42:22 UTC 2024
> Hi all,
>
> this change removes the API to allow forced allocation of new regions in G1. Before region pinning,
> G1 would force allocations of new regions beyond the calculated young gen limit while the gclocker had been active.
>
> With region pinning, this forced allocation is not necessary any more and unused.
>
> Depends on #19061.
>
> Testing: local compilation, gha
>
> Hth,
> Thomas
Thomas Schatzl has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
- Merge branch 'master' into submit/8331562-remove-force-allocation-api
- 8331562
Hi all,
this change removes the API to allow forced allocation of new regions in G1. Before region pinning,
G1 would force allocations of new regions beyond the calculated young gen limit while the gclocker
had been active.
With region pinning, this forced allocation is not necessary any more.
Testing: local compilation, gha
Hth,
Thomas
- 8331402
The is_active() call for the various G1HRPrinter methods is unnecessary as the log_trace() call will do the exact same check again anyway.
Hth,
Thomas
- 8331401
The HRPrinter does not have any member variables (static or non-static), so it can be made AllStatic.
I.e. there is no reason to have an instance variable around.
Testing: gha
Hth,
Thomas
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19062/files
- new: https://git.openjdk.org/jdk/pull/19062/files/aec80818..27d46adf
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19062&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19062&range=00-01
Stats: 3974 lines in 210 files changed: 1350 ins; 617 del; 2007 mod
Patch: https://git.openjdk.org/jdk/pull/19062.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19062/head:pull/19062
PR: https://git.openjdk.org/jdk/pull/19062
More information about the hotspot-gc-dev
mailing list