RFR: JDK-8148733: G1: Add log message to print the heap region size
Stefan Johansson
stefan.johansson at oracle.com
Mon Feb 1 15:11:02 UTC 2016
Hi Bengt,
On 2016-02-01 10:50, Bengt Rutisson wrote:
>
> Hi everyone,
>
> Could I have a couple of reviews for this small change to add a log
> message to print the heap region size?
>
> http://cr.openjdk.java.net/~brutisso/8148733/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8148733
>
Looks good,
Stefan
> Currently the heap region size is not explicitly visible in the GC log
> files. Since G1 calculates the region size at startup and then sticks
> with that size, it is enough to log the region size once during
> initialization.
>
> The new log message looks like this:
>
> [0.013s][info ][gc,heap] Heap region size: 1M
>
> The test test/gc/logging/TestDeprecatedPrintFlags.java had to be
> updated to handle the padding in the logging of the tags. Before I
> added this message there was logging where the tags looked like this
> "[gc]", but since the new logging appears early in the log file these
> log message now look like "[gc ]".
>
> Thanks,
> Bengt
>
>
More information about the hotspot-gc-dev
mailing list