# HG changeset patch # User tonyp # Date 1236442057 18000 # Node ID da1dfec1562394a4fec1617d9ca216d8e7085cdf # Parent db6a4c28a94306b51efd1293198197d4529ae9c9 6814467: G1: small fixes related to concurrent marking verboseness Summary: A few small fixes to remove some inconsistencies in the concurrent mark-related verbose GC output. Reviewed-by: jmasa diff -r db6a4c28a943 -r da1dfec15623 hotspot/src/share/vm/gc_implementation/g1/concurrentMark.cpp --- a/hotspot/src/share/vm/gc_implementation/g1/concurrentMark.cpp Sat Mar 07 11:07:37 2009 -0500 +++ b/hotspot/src/share/vm/gc_implementation/g1/concurrentMark.cpp Sat Mar 07 11:07:37 2009 -0500 @@ -420,6 +420,10 @@ _has_overflown(false), _concurrent(false), + _has_aborted(false), + _restart_for_overflow(false), + _concurrent_marking_in_progress(false), + _should_gray_objects(false), // _verbose_level set below diff -r db6a4c28a943 -r da1dfec15623 hotspot/src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp --- a/hotspot/src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp Sat Mar 07 11:07:37 2009 -0500 +++ b/hotspot/src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp Sat Mar 07 11:07:37 2009 -0500 @@ -107,7 +107,7 @@ if (PrintGC) { gclog_or_tty->date_stamp(PrintGCDateStamps); gclog_or_tty->stamp(PrintGCTimeStamps); - tty->print_cr("[GC concurrent-mark-start]"); + gclog_or_tty->print_cr("[GC concurrent-mark-start]"); } if (!g1_policy->in_young_gc_mode()) { @@ -320,8 +320,6 @@ set_in_progress(); clear_started(); if (TraceConcurrentMark) gclog_or_tty->print_cr("CM-starting"); - - return; } // Note: this method, although exported by the ConcurrentMarkSweepThread, diff -r db6a4c28a943 -r da1dfec15623 hotspot/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp --- a/hotspot/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp Sat Mar 07 11:07:37 2009 -0500 +++ b/hotspot/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp Sat Mar 07 11:07:37 2009 -0500 @@ -2332,7 +2332,6 @@ void G1CollectedHeap::checkConcurrentMark() { VerifyMarkedObjsClosure verifycl(this); - doConcurrentMark(); // MutexLockerEx x(getMarkBitMapLock(), // Mutex::_no_safepoint_check_flag); object_iterate(&verifycl);