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