[jdk8u-dev] RFR: 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash

Aleksey Shipilev shade at openjdk.org
Tue Sep 2 12:20:50 UTC 2025


On Thu, 14 Aug 2025 10:25:41 GMT, Kerem Kat <krk at openjdk.org> wrote:

> 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.

This looks fine. Like the backports to other releases, this looks like a nice safety net.

But yes, @gnu-andrew or @jerboaa would need to approve it for JDK 8.

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

Marked as reviewed by shade (Reviewer).

PR Review: https://git.openjdk.org/jdk8u-dev/pull/674#pullrequestreview-3176289185


More information about the jdk8u-dev mailing list