On Tue, 30 Jul 2024 06:46:20 GMT, Jiawei Tang <[email protected]> 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
