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