RFR: 8342818: Implement JEP 509: JFR CPU-Time Profiling [v56]
Markus Grönlund
mgronlun at openjdk.org
Wed Jun 4 17:49:28 UTC 2025
On Wed, 4 Jun 2025 17:41:46 GMT, Johannes Bechberger <jbechberger at openjdk.org> wrote:
>> src/hotspot/share/jfr/periodic/sampling/jfrCPUTimeThreadSampler.cpp line 300:
>>
>>> 298: if (!Atomic::cmpxchg(&_disenrolled, false, true)) {
>>> 299: log_trace(jfr)("Disenrolling CPU thread sampler");
>>> 300: if (Atomic::fetch_then_and(&_signal_handler_installed, false)) {
>>
>> fetch_then_and with false? Must be simpler way to express this? Like Atomic::load(&_signal_handler_installed)?
>
> I also want to set it. Wanted to do a simple ::xchg, but this doesn't exist for booleans.
1 & 0 -> 0, are you setting _signal_handler_installed to false? Why?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25302#discussion_r2127125344
More information about the serviceability-dev
mailing list