RFR: 8359348: G1: Improve cpu usage measurements for heap sizing [v5]

Ivan Walulya iwalulya at openjdk.org
Tue Aug 26 14:50:50 UTC 2025


> Hi,
> 
> Please review this patch which takes into account the impact of concurrent GC activity on mutator threads when computing the time spent on GC activity in a time interval. Previously, only the GC pause times were considered, and the overhead imposed by concurrent GC worker threads was not included.
> 
> With this change, we now estimate the impact of concurrent GC by dividing `elapsed_gc_cpu_time` by the number of CPUs. This provides an approximation of the additional time attributable to GC activity, assuming a fair CPU resource sharing.  Although this method does not account for contention on other shared resources (such as memory bandwidth or caches), it offers a reasonable estimate for most scenarios.
> 
> Testing: Tier 1

Ivan Walulya has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 15 commits:

 - Albert Review + Merge
 - Merge remote-tracking branch 'upstream/master' into ConcCPUImpact
 - remove unused
 - Merge remote-tracking branch 'upstream/master' into ConcCPUImpact
 - accumlate concurrent_gc_impact_time
 - Merge remote-tracking branch 'upstream/master' into ConcCPUImpact
 - cleanup after merge
 - Merge remote-tracking branch 'upstream/master' into ConcCPUImpact
 - Merge branch 'NormalizeTiming' into ConcCPUImpact
 - Thomas suggestion
 - ... and 5 more: https://git.openjdk.org/jdk/compare/3641c32c...bf349d2d

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

Changes: https://git.openjdk.org/jdk/pull/26351/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=26351&range=04
  Stats: 70 lines in 5 files changed: 44 ins; 1 del; 25 mod
  Patch: https://git.openjdk.org/jdk/pull/26351.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/26351/head:pull/26351

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


More information about the hotspot-gc-dev mailing list