RFR: 8350313: Include timings for leaving safepoint in safepoint logging [v2]
Xiaolong Peng
xpeng at openjdk.org
Tue Feb 25 17:12:12 UTC 2025
On Tue, 25 Feb 2025 16:54:43 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Xiaolong Peng has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Renames
>
> src/hotspot/share/runtime/safepoint.cpp line 870:
>
>> 868: jlong SafepointTracing::_last_safepoint_begin_time_ns = 0;
>> 869: jlong SafepointTracing::_last_safepoint_sync_time_ns = 0;
>> 870: jlong SafepointTracing::_last_leave_safepoint_time_ns = 0;
>
> Note: it is `_last_safepoint_...`
lol, fixed. I realized it after pushing code(wasn't feeling conformable to put verb after the noun.)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23756#discussion_r1970204051
More information about the hotspot-runtime-dev
mailing list