RFR: 8288497: add support for JavaThread::is_gc_barrier_detached()

David Holmes dholmes at openjdk.java.net
Thu Jun 16 00:06:09 UTC 2022


On Wed, 15 Jun 2022 15:50:18 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 264:
> 
>> 262: inline bool JavaThread::is_exiting() const {
>> 263:   // Use load-acquire so that setting of _terminated by
>> 264:   // 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.

It is a bogus comment - acquire semantics has nothing to do with seeing things more quickly.

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

PR: https://git.openjdk.org/jdk19/pull/20


More information about the hotspot-runtime-dev mailing list