RFR: 8265932: Move safepoint related fields from class Thread to JavaThread

Coleen Phillimore coleenp at openjdk.java.net
Tue Apr 27 12:14:37 UTC 2021


On Mon, 26 Apr 2021 15:01:41 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> Hi,
> 
> Please review the following change. A left over from 8265453 was moving _poll_data from class Thread to JavaThread. There are also other fields like _no_safepoint_count and _visited_for_critical_count that should be moved.
> Tested in mach5 tiers 1-5. Tested it builds ppc, arm and s390. Tested it builds in release, optimized and fastdebug modes.
> 
> Thanks,
> Patricio

src/hotspot/share/runtime/mutex.cpp line 453:

> 451:     if (new_owner->is_Java_thread() && _allow_vm_block && this != tty_lock) {
> 452:       new_owner->as_Java_thread()->inc_no_safepoint_count();
> 453:     }

I didn't notice this on first pass.  I like the change because it was an awkward function anyway because of the enable/disable parameter.

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

PR: https://git.openjdk.java.net/jdk/pull/3695


More information about the hotspot-runtime-dev mailing list