Re: RFR: 8296492: Remove ObjectLocker in JVMTI get_subgroups call [v6]

2022-11-17 Thread Coleen Phillimore
On Tue, 15 Nov 2022 18:52:37 GMT, Coleen Phillimore wrote: >> The JVM code took a ThreadGroup lock before poking into ThreadGroup fields. >> Call a method in the ThreadGroup to call the synchronized method instead. >> Tested with tier 1-4. > > Coleen Phillimore has updated the pull request incr

Re: RFR: 8296492: Remove ObjectLocker in JVMTI get_subgroups call [v6]

2022-11-16 Thread David Holmes
On Tue, 15 Nov 2022 18:52:37 GMT, Coleen Phillimore wrote: >> The JVM code took a ThreadGroup lock before poking into ThreadGroup fields. >> Call a method in the ThreadGroup to call the synchronized method instead. >> Tested with tier 1-4. > > Coleen Phillimore has updated the pull request incr

Re: RFR: 8296492: Remove ObjectLocker in JVMTI get_subgroups call [v6]

2022-11-16 Thread Alan Bateman
On Tue, 15 Nov 2022 18:52:37 GMT, Coleen Phillimore wrote: >> The JVM code took a ThreadGroup lock before poking into ThreadGroup fields. >> Call a method in the ThreadGroup to call the synchronized method instead. >> Tested with tier 1-4. > > Coleen Phillimore has updated the pull request incr

Re: RFR: 8296492: Remove ObjectLocker in JVMTI get_subgroups call [v6]

2022-11-16 Thread Coleen Phillimore
On Tue, 15 Nov 2022 18:52:37 GMT, Coleen Phillimore wrote: >> The JVM code took a ThreadGroup lock before poking into ThreadGroup fields. >> Call a method in the ThreadGroup to call the synchronized method instead. >> Tested with tier 1-4. > > Coleen Phillimore has updated the pull request incr

Re: RFR: 8296492: Remove ObjectLocker in JVMTI get_subgroups call [v6]

2022-11-16 Thread Coleen Phillimore
On Wed, 16 Nov 2022 03:54:31 GMT, Serguei Spitsyn wrote: >> Coleen Phillimore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Review comments. > > src/hotspot/share/prims/jvmtiEnvBase.cpp line 564: > >> 562: >> 563: for (int i = 0; i

Re: RFR: 8296492: Remove ObjectLocker in JVMTI get_subgroups call [v6]

2022-11-15 Thread Serguei Spitsyn
On Tue, 15 Nov 2022 18:52:37 GMT, Coleen Phillimore wrote: >> The JVM code took a ThreadGroup lock before poking into ThreadGroup fields. >> Call a method in the ThreadGroup to call the synchronized method instead. >> Tested with tier 1-4. > > Coleen Phillimore has updated the pull request incr

Re: RFR: 8296492: Remove ObjectLocker in JVMTI get_subgroups call [v6]

2022-11-15 Thread Coleen Phillimore
> The JVM code took a ThreadGroup lock before poking into ThreadGroup fields. > Call a method in the ThreadGroup to call the synchronized method instead. > Tested with tier 1-4. Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision: R