RFR: 8307068: store a JavaThread* in the java.lang.Thread object after the JavaThread* is added to the main ThreadsList [v4]

Andrei Pangin apangin at openjdk.org
Thu May 4 18:19:17 UTC 2023


On Tue, 2 May 2023 20:54:21 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> A small fix to store a JavaThread* in the java.lang.Thread object after the JavaThread*
>> is added to the main ThreadsList. Also adds the following:
>> - `java_lang_Thread::thread_acquire()`
>> - `java_lang_Thread::release_set_thread()`
>> 
>> And uses them on an as-needed basis.
>
> Daniel D. Daugherty has updated the pull request incrementally with one additional commit since the last revision:
> 
>   dholmes CR - update comments and switch from java_lang_Thread::set_thread() to release_set_thread().

Looks good. I only wonder if we can always use `release_set_thread` and remove plain `set_thread` entirely to avoid confusion.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/13723#issuecomment-1535209441


More information about the hotspot-runtime-dev mailing list