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