RFR: 8356648: runtime/Thread/AsyncExceptionTest.java fails with +StressCompiledExceptionHandlers [v2]
Dean Long
dlong at openjdk.org
Thu May 22 19:32:08 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.
Dean Long has updated the pull request incrementally with one additional commit since the last revision:
add -XX:+IgnoreUnrecognizedVMOptions
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/25375/files
- new: https://git.openjdk.org/jdk/pull/25375/files/52ff78a5..f7c14901
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=25375&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=25375&range=00-01
Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 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