RFR: 8342818: Implement JEP 509: JFR CPU-Time Profiling [v50]
Patricio Chilano Mateo
pchilanomate at openjdk.org
Wed Jun 4 14:59:33 UTC 2025
On Wed, 4 Jun 2025 14:52:04 GMT, Johannes Bechberger <jbechberger at openjdk.org> wrote:
>> Why would we be accessing invalid memory?
>
> Which differences to the example code are you seeing?
>
>
> sev.sigev_notify = SIGEV_SIGNAL;
> sev.sigev_signo = SIG;
> sev.sigev_value.sival_ptr = &timerid;
> if (timer_create(CLOCKID, &sev, &timerid) == -1)
> errExit("timer_create");
>
> printf("timer ID is %#jx\n", (uintmax_t) timerid);
>
> /* Start the timer. */
>
> freq_nanosecs = atoll(argv[2]);
> its.it_value.tv_sec = freq_nanosecs / 1000000000;
> its.it_value.tv_nsec = freq_nanosecs % 1000000000;
> its.it_interval.tv_sec = its.it_value.tv_sec;
> its.it_interval.tv_nsec = its.it_value.tv_nsec;
>
>
>
> Is similar to:
>
>
> ((int*)&sev.sigev_notify)[1] = thread->osthread()->thread_id();
> clockid_t clock;
> int err = pthread_getcpuclockid(thread->osthread()->pthread_id(), &clock);
> if (err != 0) {
> log_error(jfr)("Failed to get clock for thread sampling: %s", os::strerror(err));
> return false;
> }
> if (timer_create(clock, &sev, &t) < 0) {
> return false;
> }
> int64_t period = get_sampling_period();
> if (period != 0) {
> set_timer_time(t, period);
> }
The `sigev_value` member is used to pass data that you can read in the signal handler. The address of `t` won't be valid anymore once you return from this function. In that example the address of `timerid ` is still valid.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25302#discussion_r2126817165
More information about the serviceability-dev
mailing list