8143544: Make CMSCollector::is_cms_reachable() non-product

Reviewed-by: ysr, tschatzl
This commit is contained in:
Bengt Rutisson 2015-11-25 08:52:55 +01:00
parent a12ebf3b74
commit 0091d09fd2
2 changed files with 6 additions and 7 deletions

View file

@ -2240,7 +2240,6 @@ bool CMSCollector::have_cms_token() {
} }
return false; return false;
} }
#endif
// Check reachability of the given heap address in CMS generation, // Check reachability of the given heap address in CMS generation,
// treating all other generations as roots. // treating all other generations as roots.
@ -2260,21 +2259,21 @@ bool CMSCollector::is_cms_reachable(HeapWord* addr) {
// Clear the marking bit map array before starting, but, just // Clear the marking bit map array before starting, but, just
// for kicks, first report if the given address is already marked // 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"); _markBitMap.isMarked(addr) ? "" : " not");
if (verify_after_remark()) { if (verify_after_remark()) {
MutexLockerEx x(verification_mark_bm()->lock(), Mutex::_no_safepoint_check_flag); MutexLockerEx x(verification_mark_bm()->lock(), Mutex::_no_safepoint_check_flag);
bool result = verification_mark_bm()->isMarked(addr); bool result = verification_mark_bm()->isMarked(addr);
gclog_or_tty->print_cr("TransitiveMark: Address " PTR_FORMAT " %s marked", p2i(addr), tty->print_cr("TransitiveMark: Address " PTR_FORMAT " %s marked", p2i(addr),
result ? "IS" : "is NOT"); result ? "IS" : "is NOT");
return result; return result;
} else { } else {
gclog_or_tty->print_cr("Could not compute result"); tty->print_cr("Could not compute result");
return false; return false;
} }
} }
#endif
void void
CMSCollector::print_on_error(outputStream* st) { CMSCollector::print_on_error(outputStream* st) {

View file

@ -926,7 +926,7 @@ class CMSCollector: public CHeapObj<mtGC> {
// one (foreground collector or background collector). // one (foreground collector or background collector).
static void check_correct_thread_executing() PRODUCT_RETURN; 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 // Performance Counter Support
CollectorCounters* counters() { return _gc_counters; } CollectorCounters* counters() { return _gc_counters; }