On Sat, 16 Nov 2024 17:54:01 GMT, Kelvin Nilsen <kdnil...@openjdk.org> wrote:
>> I think it's here for the same reason `propagate_gc_state_to_java_threads` >> is here. Having it here makes it easier to see that this critical >> synchronization step happens for every safepoint. > > I'm inclined to leave as is, not wanting to disrupt stability of code that is > known to be working. But if there is strong preference otherwise, we can > make a change... OK, but file a bug for it to clean up. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21273#discussion_r1852524191