[jdk8u-dev] RFR: 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash [v2]
Kerem Kat
krk at openjdk.org
Tue Sep 2 15:34:08 UTC 2025
> Backport of [JDK-8212155](https://bugs.openjdk.org/browse/JDK-8212155).
>
> The patch did not apply cleanly due to a merge conflict. The original code `JvmtiThreadState::state_for(JavaThread::current())` had already been refactored to `JavaThread::current()->jvmti_thread_state()` in this branch.
>
> I have also removed the `@requires vm.jvmti` tag from the test, as it is not supported by the jtreg version in JDK 11.
>
> For JDK 8, `DynamicCodeGeneratedTest.sh` was added as the runner, which compiles the library first.
>
> The `DynamicCodeGeneratedTest` was confirmed to sometimes crash without this patch and passes with it.
Kerem Kat has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
- Merge remote-tracking branch 'upstream/master' into JDK-8212155
- Fix test for JDK 8
- Backport 64ec8b3e5c8a8d44c92591710d73b833f13c1500
-------------
Changes:
- all: https://git.openjdk.org/jdk8u-dev/pull/674/files
- new: https://git.openjdk.org/jdk8u-dev/pull/674/files/6dac15dc..bb97234c
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=674&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=674&range=00-01
Stats: 1741 lines in 98 files changed: 279 ins; 1087 del; 375 mod
Patch: https://git.openjdk.org/jdk8u-dev/pull/674.diff
Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/674/head:pull/674
PR: https://git.openjdk.org/jdk8u-dev/pull/674
More information about the jdk8u-dev
mailing list