RFR: 8288497: add support for JavaThread::is_gc_barrier_detached()
Daniel D. Daugherty
dcubed at openjdk.org
Thu Jun 16 16:17:05 UTC 2022
On Wed, 15 Jun 2022 15:51:13 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
>> A trivial fix to add support for JavaThread::is_gc_barrier_detached() which allows
>> us to add checks to detect failures like:
>>
>> JDK-8288139 JavaThread touches oop after GC barrier is detached
>> https://bugs.openjdk.org/browse/JDK-8288139
>>
>> This fix along with the fix for JDK-8288139 has been tested in Mach5 Tier[1-8].
>> There are no related failures in Mach5 Tier[1-7]; Mach5 Tier8 is still running.
>
> src/hotspot/share/runtime/thread.inline.hpp line 281:
>
>> 279: inline bool JavaThread::is_terminated() const {
>> 280: // Use load-acquire so that setting of _terminated by
>> 281: // JavaThread::set_terminated() is seen more quickly.
>
> This comment should have been updated when the code in
> JavaThread::exit() was refactored into JavaThread::set_terminated().
> I'm doing it as part of this fix for clarity.
See above discussion about this comment.
I'm in favor of dropping this comment.
-------------
PR: https://git.openjdk.org/jdk19/pull/20
More information about the hotspot-runtime-dev
mailing list