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