RFR: 8335167: Test runtime/Thread/TestAlwaysPreTouchStacks.java failed with Expected a higher ratio between stack committed and reserved [v3]

Afshin Zafari azafari at openjdk.org
Thu Sep 12 12:50:18 UTC 2024


> The test program runs once with PreTouch and another without PreTouch stacks. For each run the amounts of reserved and committed stack regions are held and then compared the ratio of with and without PreTouch. The ratio of PreTouch test should be greater than the other (because there will be higher committed regions due to pre-touching the pages).
> 
> In other words, the old version of this test ran two tests: 1) `preTouchTest` and 2) `noPreTouchTest` and extracted the amount of reserved and committed memory for stack. The ratio of the committed to reserved for preTouchTest ($ratio = \frac{committed}{reserved}$) is expected to be high (>75%) and for the noPreTouchTest is expected to be small (<50%). These expected amounts are not robust for all cases and resulted in 8335167.
> 
> In this PR, only one test is run with two sets of options for preTouch and noPreTouch stack memory. The amount of reserved and committed are stored after each run and the ratio of two runs are compared. It is expected that the preTouch has a greater ratio than the noPreTouch (`preTouch.ratio > noPreTouch.ratio`). 
> 
> ### Tests
> The specific test is run on linux-x64, windows-x64 and macosx-aarch64 in tiers1-5 (100+ times).

Afshin Zafari has updated the pull request incrementally with two additional commits since the last revision:

 - removed extra calls of the diagnostic report
 - reportDiagnosticSummary is called only at exceptions

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/20531/files
  - new: https://git.openjdk.org/jdk/pull/20531/files/71a0f1f7..002380ca

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

  Stats: 4 lines in 1 file changed: 2 ins; 1 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/20531.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20531/head:pull/20531

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


More information about the hotspot-runtime-dev mailing list