RFR: 8356648: runtime/Thread/AsyncExceptionTest.java fails with +StressCompiledExceptionHandlers [v3]
Dean Long
dlong at openjdk.org
Fri May 23 00:34:11 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:
Update test/hotspot/jtreg/runtime/Thread/AsyncExceptionTest.java
Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/25375/files
- new: https://git.openjdk.org/jdk/pull/25375/files/f7c14901..20312647
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=25375&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=25375&range=01-02
Stats: 1 line in 1 file changed: 0 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