On Tue, 30 Mar 2021 21:12:47 GMT, Richard Reingruber <rr...@openjdk.org> wrote:
> Hi Robbin,
>
> this is a great simplification. Excellent work! :)
>
> Thanks, Richard.
Thanks Richard!
> src/hotspot/share/prims/jvmtiRawMonitor.cpp line 421:
>
>> 419: guarantee(jt->thread_state() == _thread_in_native, "invariant");
>> 420: for (;;) {
>> 421: if (!SafepointMechanism::should_process(jt)) {
>
> Potential follow-up enhancement: there's a very similar for(;;) loop in
> JvmtiRawMonitor::raw_enter(). It looks as if it was possible to merge both
> loops into just one loop in simple_enter().
Yes!
-------------
PR: https://git.openjdk.java.net/jdk/pull/3191
