8143544: Make CMSCollector::is_cms_reachable() non-product
Reviewed-by: ysr, tschatzl
--- 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; }