RFR: 8361897: gc/z/TestUncommit.java fails with Uncommitted too slow

Axel Boldt-Christmas aboldtch at openjdk.org
Tue Jul 29 15:03:29 UTC 2025


This proposed change loosens the the threshold for flagging slow un-committing as an error in TestUncommit.java. Rather than requiring the un-committing be completed within 3 ZUncommitDelay, allow up to 5 ZUncommitDelay to accommodate the test being executed in environments where external factors affect whether un-committing is run as expected. 

In the observed intermittent failures, what can be seen is that the un-committer thread wakes up a significant time after the expected timeout has expired. And because we expect multiple timeouts, the accumulated extra time waiting can push the actual delay just past 3 ZUncommitDelay. Using a factor of 5 instead should remove all but the most extreme intermittent occurrences and still capture if the un-commit logic breaks completely.

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

Commit messages:
 - 8361897: gc/z/TestUncommit.java fails with Uncommitted too slow

Changes: https://git.openjdk.org/jdk/pull/26536/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=26536&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8361897
  Stats: 9 lines in 1 file changed: 9 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/26536.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/26536/head:pull/26536

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


More information about the hotspot-gc-dev mailing list