Re: RFR: 8329491: GetThreadListStackTraces function should use JvmtiHandshake [v2]

2024-04-09 Thread Serguei Spitsyn
On Tue, 9 Apr 2024 00:56:16 GMT, Leonid Mesnik wrote: >> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review: cleanup - removed temporary logging used for debugging > > src/hotspot/share/prims/jvmtiEnvBase.cpp line 207

Re: RFR: 8329491: GetThreadListStackTraces function should use JvmtiHandshake [v2]

2024-04-08 Thread Leonid Mesnik
On Tue, 2 Apr 2024 23:52:33 GMT, Serguei Spitsyn wrote: >> The internal JVM TI `JvmtiHandshake` and `JvmtiUnitedHandshakeClosure` >> classes were introduced in the JDK 22 to unify/simplify the JVM TI functions >> supporting implementation of the virtual threads. This enhancement is to >> refac

Re: RFR: 8329491: GetThreadListStackTraces function should use JvmtiHandshake [v2]

2024-04-02 Thread Serguei Spitsyn
On Tue, 2 Apr 2024 22:20:12 GMT, Patricio Chilano Mateo wrote: >> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review: cleanup - removed temporary logging used for debugging > > src/hotspot/share/prims/jvmtiEnvBase.cp

Re: RFR: 8329491: GetThreadListStackTraces function should use JvmtiHandshake [v2]

2024-04-02 Thread Serguei Spitsyn
On Tue, 2 Apr 2024 23:52:33 GMT, Serguei Spitsyn wrote: >> The internal JVM TI `JvmtiHandshake` and `JvmtiUnitedHandshakeClosure` >> classes were introduced in the JDK 22 to unify/simplify the JVM TI functions >> supporting implementation of the virtual threads. This enhancement is to >> refac

Re: RFR: 8329491: GetThreadListStackTraces function should use JvmtiHandshake [v2]

2024-04-02 Thread Serguei Spitsyn
> The internal JVM TI `JvmtiHandshake` and `JvmtiUnitedHandshakeClosure` > classes were introduced in the JDK 22 to unify/simplify the JVM TI functions > supporting implementation of the virtual threads. This enhancement is to > refactor the JVM TI function `GetThreadListStackTraces` on the base