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,

> 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