RFR: 8342818: Implement JEP 509: JFR CPU-Time Profiling [v42]
Johannes Bechberger
jbechberger at openjdk.org
Wed Jun 4 06:34:30 UTC 2025
On Wed, 4 Jun 2025 00:28:46 GMT, Andrei Pangin <apangin at openjdk.org> wrote:
>> Johannes Bechberger has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Rename autoadapt
>
> src/hotspot/share/jfr/support/jfrThreadLocal.cpp line 558:
>
>> 556: void JfrThreadLocal::set_cpu_timer(timer_t* timer) {
>> 557: if (_cpu_timer == nullptr) {
>> 558: _cpu_timer = JfrCHeapObj::new_array<timer_t>(1);
>
> `timer_t` is a primitive type, at most one machine word. Why extra indirection and allocation?
@mgronlun wanted this indirection to move it abstract from implementation details
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25302#discussion_r2125758074
More information about the hotspot-dev
mailing list