Print this page
rev 7125 : 7176220: 'Full GC' events miss date stamp information occasionally
Summary: Move date stamp logic into GCTraceTime
Reviewed-by: brutisso, tschatzl

Split Split Close
Expand all
Collapse all
          --- old/hotspot/src/share/vm/gc_implementation/parallelScavenge/psMarkSweep.cpp
          +++ new/hotspot/src/share/vm/gc_implementation/parallelScavenge/psMarkSweep.cpp
↓ open down ↓ 159 lines elided ↑ open up ↑
 160  160    heap->pre_full_gc_dump(_gc_timer);
 161  161  
 162  162    // Filled in below to track the state of the young gen after the collection.
 163  163    bool eden_empty;
 164  164    bool survivors_empty;
 165  165    bool young_gen_empty;
 166  166  
 167  167    {
 168  168      HandleMark hm;
 169  169  
 170      -    gclog_or_tty->date_stamp(PrintGC && PrintGCDateStamps);
 171  170      TraceCPUTime tcpu(PrintGCDetails, true, gclog_or_tty);
 172  171      GCTraceTime t1(GCCauseString("Full GC", gc_cause), PrintGC, !PrintGCDetails, NULL, _gc_tracer->gc_id());
 173  172      TraceCollectorStats tcs(counters());
 174  173      TraceMemoryManagerStats tms(true /* Full GC */,gc_cause);
 175  174  
 176  175      if (TraceGen1Time) accumulated_time()->start();
 177  176  
 178  177      // Let the size policy know we're starting
 179  178      size_policy->major_collection_begin();
 180  179  
↓ open down ↓ 502 lines elided ↑ open up ↑
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX