Review request (S): 7176220: 'Full GC' events miss date stamp information occasionally
andreas.eriksson at oracle.com
Fri Sep 28 10:01:33 UTC 2012
Can I have some reviews for the following change please:
This fixes GC logs from genCollectedHeap.cpp where sometimes it would
not print date-stamps.
The fix moves printing of date-stamps into TraceTime, where time-stamps
are already printed from.
Bug with flags -XX:+UseConcMarkSweepGC -XX:+UseParNewGC -verbose:gc
2012-09-28T11:41:38.687+0200: 0.205: [Full GC 8702K->1012K(9920K),
1.253: [Full GC 3326K->1012K(9920K), 0.0105750 secs]
2.299: [Full GC 7932K->1012K(9920K), 0.0100470 secs]
2012-09-28T11:41:41.813+0200: 3.331: [Full GC 8702K->1012K(9920K),
Another route that was investigated was to add the date-stamp call
directly in genCollectedHeap.cpp, but because of complex interactions
when a full GC was initiated from a non-full GC this path was not taken.
This will slightly change the logging output when using -XX:+PrintGCDetails.
Now date-stamps will also be printed inside other lines, as time-stamps
2012-09-28T11:29:16.815+0200: 0.177: [GC0.177: [ParNew:
2710K->46K(3072K), 0.0053590 secs] 3480K->1835K(9920K), 0.0055500 secs]
[Times: user=0.01 sys=0.00, real=0.01 secs]
2012-09-28T11:29:33.260+0200: 0.184: [GC2012-09-28T11:29:33.261+0200:
0.184: [ParNew: 2642K->45K(3072K), 0.0095770 secs] 3412K->1828K(9920K),
0.0097520 secs] [Times: user=0.01 sys=0.00, real=0.01 secs]
Thanks to Bengt Rutisson for helping with the change and webrev.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-gc-dev