On Tue, 30 Jul 2024 11:20:48 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> 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
>
> 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()`.
> So, the lines 1093-1095 need to be removed now.

Change it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20373#discussion_r1699334445

Reply via email to