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

Thomas Stuefe stuefe at openjdk.org
Thu Jul 4 07:46:35 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.

Thomas Stuefe 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 seven additional commits since the last revision:

 - Merge branch 'master' into JDK-8334513-New-test-gc-TestAlwaysPreTouchBehavior-java-is-failing
 - comments for albert
 - Update TestAlwaysPreTouchBehavior.java
 - tweaks
 - fixes
 - Merge branch 'master' into JDK-8334513-New-test-gc-TestAlwaysPreTouchBehavior-java-is-failing
 - fix

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/19803/files
  - new: https://git.openjdk.org/jdk/pull/19803/files/19ed5833..a25bfca4

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

  Stats: 14736 lines in 498 files changed: 9695 ins; 3004 del; 2037 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-gc-dev mailing list