On Sat, 26 Oct 2024 05:13:08 GMT, Alan Bateman <al...@openjdk.org> wrote:

>> src/java.base/share/classes/jdk/internal/vm/annotation/JvmtiMountTransition.java
>>  line 38:
>> 
>>> 36:  *
>>> 37:  * @implNote
>>> 38:  * This annotation is only used for the VirtualThread notifyJvmti*  
>>> methods.
>> 
>> What about VirtualThread.switchToCarrierThread and 
>> VirtualThread.switchToVirtualThread ? They also have the annotation.
>
> We working to remove these two methods. I think the main thing for 
> JvmtiMountTransition is that the interface description provides enough 
> information to know when the annotation is needed.

Good catch, thanks. These two methods are impacted by temporary VTMS 
transitions.
I've restored as it was before.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21397#discussion_r1817696831

Reply via email to