Integrated: 8273251: Call check_possible_safepoint() from SafepointMechanism::process_if_requested()

Patricio Chilano Mateo pchilanomate at openjdk.java.net
Thu Sep 2 21:26:33 UTC 2021


On Wed, 1 Sep 2021 20:53:46 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> Hi,
> 
> Please review this small fix to call check_possible_safepoint() from SafepointMechanism::process_if_requested() and remove the same call from transition_and_process(). See the bug description for more detail.
> Since there were only two lines left of common code from transition_from_native() and transition_from_vm() I just removed transition_and_process() altogether. This also saves an unneeded extra transition to _thread_in_vm when transitioning from vm to Java.
> 
> Testing tiers 1-6.
> 
> Thanks,
> Patricio

This pull request has now been integrated.

Changeset: 92b05fe0
Author:    Patricio Chilano Mateo <pchilanomate at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/92b05fe0f41b91aa88e77473725ae92ee13b052f
Stats:     34 lines in 3 files changed: 13 ins; 17 del; 4 mod

8273251: Call check_possible_safepoint() from SafepointMechanism::process_if_requested()

Reviewed-by: coleenp, dholmes

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

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


More information about the hotspot-runtime-dev mailing list