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

Axel Boldt-Christmas aboldtch at openjdk.org
Thu Jul 31 13:11:00 UTC 2025


On Tue, 29 Jul 2025 14:58:09 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: 3f21c8bd
Author:    Axel Boldt-Christmas <aboldtch at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/3f21c8bd1f0cd7f27df940505b249ba7798940cf
Stats:     9 lines in 1 file changed: 9 ins; 0 del; 0 mod

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

Reviewed-by: stefank, jsikstro

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

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


More information about the hotspot-gc-dev mailing list