On Tue, 11 May 2021 17:02:44 GMT, Daniel D. Daugherty <dcu...@openjdk.org>
wrote:
>> Robbin Ehn has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains two additional
>> commits since the last revision:
>>
>> - Merge branch 'master' into 8265753
>> - Removed manual transitions
>
> src/hotspot/share/prims/jvmtiRawMonitor.cpp line 320:
>
>> 318: // JavaThreads will enter here with state _thread_in_native.
>> 319: void JvmtiRawMonitor::raw_enter(Thread* self) {
>> 320: // TODO Atomic::load on _owner field
>
> Why is this still a TODO?
We have ~15 plain loads of _owner, I thought it would be nice to fix all at the
same time and I did not want to clobber this with that also.
I can do a follow-up and change them.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3875
