RFR: JDK-8284758: [linux] improve print_container_info [v2]

Matthias Baesken mbaesken at openjdk.java.net
Wed Apr 13 14:17:53 UTC 2022


On Wed, 13 Apr 2022 11:09:17 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   adjust print_container_helper
>
> src/hotspot/os/linux/os_linux.cpp line 2189:
> 
>> 2187: 
>> 2188: static void print_container_helper(outputStream* st, jlong j, const char* header) {
>> 2189:   st->print("%s", header);
> 
> Small nit: I' d probably add ": " here and remove those out of the caller arguments.

True, this should be changed.
I also adjusted the parameter name of print_container_helper from header to metrics because header was a bit misleading.

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

PR: https://git.openjdk.java.net/jdk/pull/8217


More information about the hotspot-runtime-dev mailing list