[jdk17] RFR: 8269240: java/foreign/stackwalk/TestAsyncStackWalk.java test failed with concurrent GC [v3]
Jorn Vernee
jvernee at openjdk.java.net
Thu Jul 15 15:54:50 UTC 2021
> This patch rewrites the prologue and epilogue of panama upcalls, in order to fix the test failure from the title.
>
> Previously, we did a call to potentially attach the current thread to the VM, and then afterwards did the same suspend and stack reguard checks that we do on the back-edge of a native downcall. Then, on the back edge of the upcall we did another conditional call to detach the thread.
>
> The suspend and reguard checks on the front-edge are incorrect, so I've changed the 2 calls to mimic what is done by JavaCallWrapper instead (with attach and detach included), and removed the old suspend and stack reguard checks.
>
> FWIW, this removes the JavaFrameAnchor save/restore MacroAssembler code. This is now written in C++. Also, MacroAssembler code was added to save/restore the result of the upcall around the call on the back-edge, which was previously missing. Since the new code allocates a handle block as well, I've added handling for those oops to frame & OptimizedUpcallBlob.
>
> Testing: local running of `jdk_foreign` on Windows and Linux (WSL). Tier 1-3
Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
Address David's review comments
-------------
Changes:
- all: https://git.openjdk.java.net/jdk17/pull/149/files
- new: https://git.openjdk.java.net/jdk17/pull/149/files/211bf316..128f48db
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk17&pr=149&range=02
- incr: https://webrevs.openjdk.java.net/?repo=jdk17&pr=149&range=01-02
Stats: 20 lines in 3 files changed: 0 ins; 8 del; 12 mod
Patch: https://git.openjdk.java.net/jdk17/pull/149.diff
Fetch: git fetch https://git.openjdk.java.net/jdk17 pull/149/head:pull/149
PR: https://git.openjdk.java.net/jdk17/pull/149
More information about the hotspot-gc-dev
mailing list