RFR: 8265753: Remove manual JavaThread transitions to blocked [v2]

Robbin Ehn rehn at openjdk.java.net
Wed May 12 06:06:01 UTC 2021


On Tue, 11 May 2021 17:02:44 GMT, Daniel D. Daugherty <dcubed at 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


More information about the serviceability-dev mailing list