Integrated: 8265984: Concurrent GC: Some tests fail "assert(is_frame_safe(f)) failed: Frame must be safe"

[Per Liden]

On Fri, 30 Apr 2021 11:13:10 GMT, Per Liden <pliden at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: 194bceca
Author:    Per Liden <pliden at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/194bceca3a4d13d4528b86359ee9d5eead3ce7ac
Stats:     5 lines in 2 files changed: 3 ins; 0 del; 2 mod

8265984: Concurrent GC: Some tests fail "assert(is_frame_safe(f)) failed: Frame must be safe"

Reviewed-by: eosterlund, stefank, zgu

-------------

PR: https://git.openjdk.java.net/jdk/pull/3814