On Wed, 29 Mar 2023 22:15:35 GMT, Patricio Chilano Mateo 
<pchilanom...@openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   review: updated correction of jt->jvmti_thread_state() links in 
>> VM_SetNotifyJvmtiEventsMode
>
> Hi Serguei,
> 
> I took a look at the patch and looks good to me. I have a couple of comments 
> though.
> 
> Thanks,
> Patricio

> @pchilano I've pushed an update where the VTMS transition bit in the 
> VirtualThread object is laways/unconditionally set. I do not see any 
> performance impact with this. Please, let me know about your opinion.
>
Latest changes look good to me. Thanks Serguei!

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

PR Comment: https://git.openjdk.org/jdk/pull/13133#issuecomment-1494901707

Reply via email to