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

[Patricio Chilano Mateo]

On Thu, 2 Sep 2021 07:58:43 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> I already run tiers 1-3 and I'm currently running tiers 4-6. Since we already include macosx-aarch64 tasks in those tiers I didn't run any additional tests. I will run an additional test cycle macosx-aarch64 specific. I'm not expecting any failures in that area though since those other places are methods with conditional statements where in one branch we might directly call process_if_requested() already due to some _safepoint_check_always lock. Caller method check_for_valid_safepoint_state() should also be safe since it asserts we are in _thread_in_vm implying WXWrite(in any case the callers of that method could also directly call process_if_requested()). We should probably merge this two methods after I fix callers of process_if_requested() to be in _thread_in_vm.
>
> Maybe `check_safepoint_ready` for the new name? (Not needed for this PR though.)

Sounds good, I'll address that in another RFE then.

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

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