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

2024-04-10 Thread Serguei Spitsyn
On Wed, 10 Apr 2024 03:17:32 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 >> refa

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

2024-04-10 Thread Leonid Mesnik
On Wed, 10 Apr 2024 03:17:32 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 >> refa

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

2024-04-09 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

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

Re: RFR: 8329491: GetThreadListStackTraces function should use JvmtiHandshake

2024-04-02 Thread Patricio Chilano Mateo
On Tue, 2 Apr 2024 08:13:20 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 > refactor

RFR: 8329491: GetThreadListStackTraces function should use JvmtiHandshake

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 of `Jvm