RFR: 8337789: JEP 509: JFR CPU-Time Profiling (Experimental) [v47]

[Andrei Pangin]

On Thu, 15 May 2025 13:18:17 GMT, Johannes Bechberger <jbechberger at openjdk.org> wrote:

>> src/hotspot/share/jfr/periodic/sampling/jfrCPUTimeThreadSampler.cpp line 887:
>> 
>>> 885:   sev.sigev_signo = SIG;
>>> 886:   sev.sigev_value.sival_ptr = &t;
>>> 887:   ((int*)&sev.sigev_notify)[1] = thread->osthread()->thread_id();
>> 
>> Is there a reason why not use `sev._sigev_un._tid` directly if this is Linux only?
>
> Good catch.

Are you sure the field is exposed in headers on all Linux distributions?
IIRC, it was not on Alpine.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20752#discussion_r2091199675