RFR: 8337331: crash: pinned virtual thread will lead to jvm crash when running with the javaagent option [v4]

[Serguei Spitsyn]

On Tue, 30 Jul 2024 06:46:20 GMT, Jiawei Tang <jwtang at openjdk.org> wrote:

>> I add the testcase which can reproduce the crash. I hope that I could get some advise if the codes need changing.
>
> Jiawei Tang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   refactor testcase and change the location of fix codes

Changes requested by sspitsyn (Reviewer).

src/hotspot/share/prims/jvmtiExport.cpp line 1098:

> 1096:   if (JavaThread::current()->is_in_any_VTMS_transition()) {
> 1097:     return false; // no events should be posted if thread is in any VTMS transition
> 1098:   }

Sorry, I was not clear the 3 lines above 1093-1095 had to be replaced with new lines 1096-1098.
The check for `is_in_any_VTMS_transition()` includes the checks `is_in_tmp_VTMS_transition()`.

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

PR Review: https://git.openjdk.org/jdk/pull/20373#pullrequestreview-2207409392
PR Review Comment: https://git.openjdk.org/jdk/pull/20373#discussion_r1696791636