# HG changeset patch # User tonyp # Date 1236442057 18000 # Node ID 7ea5ca260b28009116fc349ed677cad861296f80 # Parent ae1579717a5711a6f00f4776a0f87ee4b22f25b6 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 ae1579717a57 -r 7ea5ca260b28 src/share/vm/gc_implementation/g1/concurrentMark.cpp --- a/src/share/vm/gc_implementation/g1/concurrentMark.cpp Sat Mar 07 11:07:37 2009 -0500 +++ b/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 ae1579717a57 -r 7ea5ca260b28 src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp --- a/src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp Sat Mar 07 11:07:37 2009 -0500 +++ b/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 ae1579717a57 -r 7ea5ca260b28 src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp --- a/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp Sat Mar 07 11:07:37 2009 -0500 +++ b/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);