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