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

Daniel D. Daugherty dcubed at openjdk.org
Thu May 4 18:49:17 UTC 2023


On Thu, 4 May 2023 05:10:43 GMT, Robbin Ehn <rehn at openjdk.org> wrote:

>> 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, thanks!

@robehn and @apangin - thanks for the re-reviews!

@apangin:
>  I only wonder if we can always use release_set_thread and remove plain set_thread entirely to avoid confusion.

I think I would prefer to leave that to another PR instead of this one.

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

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


More information about the hotspot-runtime-dev mailing list