On Wed, 4 Jun 2025 19:47:17 GMT, Markus Grönlund <mgron...@openjdk.org> wrote:
>> Johannes Bechberger has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Improve > > src/hotspot/share/jfr/periodic/sampling/jfrCPUTimeThreadSampler.cpp line 634: > >> 632: >> 633: void JfrCPUSamplerThread::allow_signal_handlers() { >> 634: Atomic::release_store(&_initialize_active_signal_handler_counter, >> (u4)0); > > I think we might be getting a bit tired. I meant the function name, not the > variable name, which now looks completely bonkers :-) Well, I just misunderstood you :D ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25302#discussion_r2127329186