RFR: 8356648: runtime/Thread/AsyncExceptionTest.java fails with +StressCompiledExceptionHandlers
Dean Long
dlong at openjdk.org
Thu May 22 00:18:05 UTC 2025
The test fails (or times out) because Runtime1::counter_overflow offers a safepoint, allowing async exceptions, but there is no exception handler for the out-of-line slow path call to Runtime1::counter_overflow. Rather than add an exception handler, the simple fix is to replace JRT_BLOCK with JRT_BLOCK_NO_ASYNC so that counter_overflow doesn't need to deal with async exceptions. The GC poll point for backwards branches is sufficient to allow async exceptions.
I also improved the test so that it fails more quickly rather than waiting forever and eventually timing out, and added a C1-specific run using -XX:+StressCompiledExceptionHandlers.
-------------
Commit messages:
- improve test
- use JRT_BLOCK_NO_ASYNC
Changes: https://git.openjdk.org/jdk/pull/25375/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=25375&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8356648
Stats: 36 lines in 2 files changed: 9 ins; 11 del; 16 mod
Patch: https://git.openjdk.org/jdk/pull/25375.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25375/head:pull/25375
PR: https://git.openjdk.org/jdk/pull/25375
More information about the hotspot-dev
mailing list