RFR: 8265984: Concurrent GC: Some tests fail "assert(is_frame_safe(f)) failed: Frame must be safe"
Per Liden
pliden at openjdk.java.net
Fri Apr 30 13:18:20 UTC 2021
JDK-8265702 introduced a change in `OptoRuntime::handle_exception_C()` where it no longer processed frames when asking for the caller. This change turned out to cause intermittent test failures. We shouldn't have to process frames here, since we're not touching any oops, but it seems that processing frames here avoids some other issue. To avoid noise in out testing I suggest we revert to processing frames here for now (and add the required ThreadWXEnable for macos/aarch64), while we continue to investigate why this is needed.
Testing: Passed tier1-7 on linux/x86 and macos/aarch64
-------------
Commit messages:
- 8265984: Concurrent GC: Some tests fail "assert(is_frame_safe(f)) failed: Frame must be safe"
Changes: https://git.openjdk.java.net/jdk/pull/3814/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3814&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8265984
Stats: 5 lines in 2 files changed: 3 ins; 0 del; 2 mod
Patch: https://git.openjdk.java.net/jdk/pull/3814.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3814/head:pull/3814
PR: https://git.openjdk.java.net/jdk/pull/3814
More information about the hotspot-gc-dev
mailing list