RFR: 8360641: TestCompilerCounts fails after 8354727

Damon Fenacci dfenacci at openjdk.org
Mon Jun 30 06:53:44 UTC 2025


On Fri, 27 Jun 2025 18:09:23 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:

> After integrating #25872 the calculation of the`CICompilerCount` ergonomic became dependent on the size of `NonNMethodCodeHeapSize`, which itself is an ergonomic based on the available memory. Thus, depending on the system, the test `compiler/arguments/TestCompilerCounts.java` failed, i.e. locally this failed, but not on CI servers.
> 
> This PR changes the test to reflect the changes introduced in #25872.
> 
> Testing:
>  - [ ] [Github Actions](https://github.com/mhaessig/jdk/actions/runs/15932906313)
>  - [ ] tier1,tier2 plus Oracle internal testing

Thanks a lot for fixing this @mhaessig!
I left a couple of inline comments and just noticed you might want to add the copyright note as well 🙂

test/hotspot/jtreg/compiler/arguments/TestCompilerCounts.java line 138:

> 136:             // Non-tiered modes
> 137:             int c1OnlyCount = heuristicCount(cpus, Compilation.C1Only);
> 138:             pass(c1OnlyCount, opt, "-XX:TieredStopAtLevel=1", "-XX:NonNMethodCodeHeapSize="+NonNMethodCodeHeapSize, "-XX:CodeCacheMinimumUseSpace="+CodeCacheMinimumUseSpace);

Very minor style thing:  maybe we can put whitespaces around the `+` signs.

test/hotspot/jtreg/compiler/arguments/TestCompilerCounts.java line 164:

> 162: 
> 163:     // Buffer sizes for caclulating the maximum number of compiler threads.
> 164:     static final int NonNMethodCodeHeapSize = 5 * 1024 * 1024;

Is the `NonNMethodCodeHeapSize` value chosen "on purpose"?

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

PR Review: https://git.openjdk.org/jdk/pull/26024#pullrequestreview-2970074252
PR Review Comment: https://git.openjdk.org/jdk/pull/26024#discussion_r2174335731
PR Review Comment: https://git.openjdk.org/jdk/pull/26024#discussion_r2174345082


More information about the hotspot-compiler-dev mailing list