RFR(s): 8197993: Remove last use of JavaThread::flush_barrier_queues()
Erik Österlund
erik.osterlund at oracle.com
Thu Feb 15 11:30:28 UTC 2018
Oops, I sent this email to the wrong thread. I will re-send it in the
correct thread.
/Erik
On 2018-02-15 11:14, Erik Österlund wrote:
> Hi Per,
>
> I wonder how we feel about the name on_thread_create in particular. It
> is crucial that this is performed not just when creating the thread,
> but particularly when adding it to the thread list while holding the
> Threads_lock to block safepoints, otherwise the G1 barriers would blow
> up. Therefore I would have preferred the name on_thread_add instead of
> on_thread_create. Would you agree with that?
>
> Thanks,
> /Erik
>
> On 2018-02-15 11:05, Erik Österlund wrote:
>> Hi Per,
>>
>> Looks good.
>>
>> /Erik
>>
>> On 2018-02-15 10:33, Per Liden wrote:
>>> With the introduction of BarrierSet::flush_deferred_barriers()
>>> there's no reason to keep JavaThread::flush_barrier_queues() around
>>> anymore. Replace the last call-site with a call to the barrier set
>>> instead.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8197993
>>> Webrev: http://cr.openjdk.java.net/~pliden/8197993/webrev.0/
>>>
>>> /Per
>>
>
More information about the hotspot-gc-dev
mailing list