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