RFR: JDK-8293114: JVM should trim the native heap [v4]

Thomas Stuefe stuefe at openjdk.org
Fri Jul 7 11:38:56 UTC 2023


On Thu, 6 Jul 2023 18:50:45 GMT, Robbin Ehn <rehn at openjdk.org> wrote:

>> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   last cleanups and shade feedback
>
> src/hotspot/share/runtime/trimNative.cpp line 137:
> 
>> 135:     os::size_change_t sc;
>> 136:     Ticks start = Ticks::now();
>> 137:     log_debug(trim)("Trim native heap started...");
> 
> TraceTime not a good fit?

I'll revert to using elapsedTime for all timing needs here.

TraceTime seems too much hassle. I want the time combined with other output in one line, and I don't want to provide a TraceTimerLogPrintFunc.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14781#discussion_r1255690649


More information about the serviceability-dev mailing list