RFR: 8334513: New test gc/TestAlwaysPreTouchBehavior.java is failing

Thomas Stuefe stuefe at openjdk.org
Sun Jun 23 11:33:34 UTC 2024


See JBS issue.

It is not completely obvious what the problem is in Oracle's CI, but the current assumption is that RSS of the testee VM gets reduced after it started and before we measured due to memory pressure.

The patch:
- exposes os::available_memory via Whitebox
- For the test to count as failed, we require a certain minimum size of available memory both before and during the start of the testee JVM. Otherwise, we throw a `SkippedException`

I have some misgivings about this solution, though:
1) obviously, it is not bullet-proof either, since it is vulnerable to fast changes in machine memory load. 
2) On MacOS, we have the problem that 'os::available_memory()' totally underreports how much memory is available. Therefore, as an estimate of whether the test is valid, it is too conservative. I opened https://bugs.openjdk.org/browse/JDK-8334767 to track that issue. As long as it is not fixed, the tests will likely fall below the threshold on MacOS and, therefore, be skipped. Still, this is somewhat better than outright excluding the test for MacOS (or is it? Open to opinions)
3) `SkippedException` leads to the test counting as "passed", not "skipped". I think that is a usability issue with jtreg. I cannot easily see which tests had been skipped due to SkippedException.

Despite my doubts, I think this is the best we can come up with if we want to have such a test.

Note: One way to go about (3) would be to make "minimum available memory" a `@requires` tag, similar to os.maxMemory. However, I fear that this may be easily misused and cause many tests to be excluded without notice.

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

Commit messages:
 - tweaks
 - fixes
 - Merge branch 'master' into JDK-8334513-New-test-gc-TestAlwaysPreTouchBehavior-java-is-failing
 - fix

Changes: https://git.openjdk.org/jdk/pull/19803/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=19803&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8334513
  Stats: 108 lines in 5 files changed: 69 ins; 18 del; 21 mod
  Patch: https://git.openjdk.org/jdk/pull/19803.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/19803/head:pull/19803

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


More information about the hotspot-dev mailing list