8143544: Make CMSCollector::is_cms_reachable() non-product
authorbrutisso
Wed, 25 Nov 2015 08:52:55 +0100
changeset 34293 4c3fe9d26a2a
parent 34284 32543d155516
child 34294 a5c72bec963e
8143544: Make CMSCollector::is_cms_reachable() non-product Reviewed-by: ysr, tschatzl
hotspot/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp
hotspot/src/share/vm/gc/cms/concurrentMarkSweepGeneration.hpp
--- a/hotspot/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp	Wed Nov 25 04:51:32 2015 +0000
+++ b/hotspot/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp	Wed Nov 25 08:52:55 2015 +0100
@@ -2240,7 +2240,6 @@
   }
   return false;
 }
-#endif
 
 // Check reachability of the given heap address in CMS generation,
 // treating all other generations as roots.
@@ -2260,21 +2259,21 @@
 
   // Clear the marking bit map array before starting, but, just
   // for kicks, first report if the given address is already marked
-  gclog_or_tty->print_cr("Start: Address " PTR_FORMAT " is%s marked", p2i(addr),
+  tty->print_cr("Start: Address " PTR_FORMAT " is%s marked", p2i(addr),
                 _markBitMap.isMarked(addr) ? "" : " not");
 
   if (verify_after_remark()) {
     MutexLockerEx x(verification_mark_bm()->lock(), Mutex::_no_safepoint_check_flag);
     bool result = verification_mark_bm()->isMarked(addr);
-    gclog_or_tty->print_cr("TransitiveMark: Address " PTR_FORMAT " %s marked", p2i(addr),
-                           result ? "IS" : "is NOT");
+    tty->print_cr("TransitiveMark: Address " PTR_FORMAT " %s marked", p2i(addr),
+                  result ? "IS" : "is NOT");
     return result;
   } else {
-    gclog_or_tty->print_cr("Could not compute result");
+    tty->print_cr("Could not compute result");
     return false;
   }
 }
-
+#endif
 
 void
 CMSCollector::print_on_error(outputStream* st) {
--- a/hotspot/src/share/vm/gc/cms/concurrentMarkSweepGeneration.hpp	Wed Nov 25 04:51:32 2015 +0000
+++ b/hotspot/src/share/vm/gc/cms/concurrentMarkSweepGeneration.hpp	Wed Nov 25 08:52:55 2015 +0100
@@ -926,7 +926,7 @@
   // one (foreground collector or background collector).
   static void check_correct_thread_executing() PRODUCT_RETURN;
 
-  bool is_cms_reachable(HeapWord* addr);
+  NOT_PRODUCT(bool is_cms_reachable(HeapWord* addr);)
 
   // Performance Counter Support
   CollectorCounters* counters()    { return _gc_counters; }