8252089: Remove psParallelCompact internal debug counters
Reviewed-by: sjohanss, tschatzl
This commit is contained in:
parent
4b7f027407
commit
0257ac1447
@ -77,11 +77,6 @@ ParMarkBitMap::initialize(MemRegion covered_region)
|
||||
return false;
|
||||
}
|
||||
|
||||
#ifdef ASSERT
|
||||
extern size_t mark_bitmap_count;
|
||||
extern size_t mark_bitmap_size;
|
||||
#endif // #ifdef ASSERT
|
||||
|
||||
bool
|
||||
ParMarkBitMap::mark_obj(HeapWord* addr, size_t size)
|
||||
{
|
||||
@ -90,8 +85,6 @@ ParMarkBitMap::mark_obj(HeapWord* addr, size_t size)
|
||||
const idx_t end_bit = addr_to_bit(addr + size - 1);
|
||||
bool end_bit_ok = _end_bits.par_set_bit(end_bit);
|
||||
assert(end_bit_ok, "concurrency problem");
|
||||
DEBUG_ONLY(Atomic::inc(&mark_bitmap_count));
|
||||
DEBUG_ONLY(Atomic::add(&mark_bitmap_size, size));
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
|
@ -411,13 +411,6 @@ print_initial_summary_data(ParallelCompactData& summary_data,
|
||||
}
|
||||
#endif // #ifndef PRODUCT
|
||||
|
||||
#ifdef ASSERT
|
||||
size_t add_obj_count;
|
||||
size_t add_obj_size;
|
||||
size_t mark_bitmap_count;
|
||||
size_t mark_bitmap_size;
|
||||
#endif // #ifdef ASSERT
|
||||
|
||||
ParallelCompactData::ParallelCompactData() :
|
||||
_region_start(NULL),
|
||||
DEBUG_ONLY(_region_end(NULL) COMMA)
|
||||
@ -538,9 +531,6 @@ void ParallelCompactData::add_obj(HeapWord* addr, size_t len)
|
||||
// end_region is inclusive
|
||||
const size_t end_region = (obj_ofs + len - 1) >> Log2RegionSize;
|
||||
|
||||
DEBUG_ONLY(Atomic::inc(&add_obj_count);)
|
||||
DEBUG_ONLY(Atomic::add(&add_obj_size, len);)
|
||||
|
||||
if (beg_region == end_region) {
|
||||
// All in one region.
|
||||
_region_data[beg_region].add_live_obj(len);
|
||||
@ -992,9 +982,6 @@ void PSParallelCompact::pre_compact()
|
||||
_space_info[from_space_id].set_space(heap->young_gen()->from_space());
|
||||
_space_info[to_space_id].set_space(heap->young_gen()->to_space());
|
||||
|
||||
DEBUG_ONLY(add_obj_count = add_obj_size = 0;)
|
||||
DEBUG_ONLY(mark_bitmap_count = mark_bitmap_size = 0;)
|
||||
|
||||
// Increment the invocation count
|
||||
heap->increment_total_collections(true);
|
||||
|
||||
@ -1611,19 +1598,6 @@ void PSParallelCompact::summary_phase(ParCompactionManager* cm,
|
||||
{
|
||||
GCTraceTime(Info, gc, phases) tm("Summary Phase", &_gc_timer);
|
||||
|
||||
#ifdef ASSERT
|
||||
log_develop_debug(gc, marking)(
|
||||
"add_obj_count=" SIZE_FORMAT " "
|
||||
"add_obj_bytes=" SIZE_FORMAT,
|
||||
add_obj_count,
|
||||
add_obj_size * HeapWordSize);
|
||||
log_develop_debug(gc, marking)(
|
||||
"mark_bitmap_count=" SIZE_FORMAT " "
|
||||
"mark_bitmap_bytes=" SIZE_FORMAT,
|
||||
mark_bitmap_count,
|
||||
mark_bitmap_size * HeapWordSize);
|
||||
#endif // ASSERT
|
||||
|
||||
// Quick summarization of each space into itself, to see how much is live.
|
||||
summarize_spaces_quick();
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user