RFR: 8253173: Print heap before and after GC lacks a newline

Roman Kennke rkennke at openjdk.java.net
Tue Sep 15 15:50:48 UTC 2020


On Tue, 15 Sep 2020 15:04:09 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> When GC logging was converted to Unified Logging a newline went missing.
> 
> -  st->print_cr("{Heap before GC invocations=%u (full %u):",
> -               heap()->total_collections(),
> -               heap()->total_full_collections());
> vs
> +  LogHandle(gc, heap) log;
> +  if (log.is_trace()) {
> +    log.trace("Heap before GC invocations=%u (full %u):", heap()->total_collections(),
> heap()->total_full_collections()); This gives lines like this:
> GC(29) Heap after GC invocations=30 (full 30): ZHeap           used 20M, capacity 2014M, max capacity 30718M
> GC(29)  Metaspace       used 6779K, capacity 6857K, committed 7040K, reserved 1056768K
> GC(29)   class space    used 574K, capacity 620K, committed 640K, reserved 1048576K
> Instead of:
> GC(29) Heap after GC invocations=30 (full 30):
> GC(29) ZHeap           used 20M, capacity 2014M, max capacity 30718M
> GC(29)  Metaspace       used 6779K, capacity 6857K, committed 7040K, reserved 1056768K
> GC(29)   class space    used 574K, capacity 620K, committed 640K, reserved 1048576K
> In our hs_err files we print this with a newline:
>   
> st.print_cr("{Heap %s GC invocations=%u (full %u):",
>             before ? "before" : "after",
>             heap->total_collections(),
>             heap->total_full_collections());
> I propse that we add this missing newline.

Looks good to me.

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

Marked as reviewed by rkennke (Reviewer).

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



More information about the hotspot-gc-dev mailing list