--- old/src/share/vm/gc/g1/g1ConcurrentMark.cpp 2017-03-01 15:41:29.779389990 -0500 +++ new/src/share/vm/gc/g1/g1ConcurrentMark.cpp 2017-03-01 15:41:28.507316651 -0500 @@ -414,7 +414,6 @@ _task_queues(new G1CMTaskQueueSet((int) _max_worker_id)), _terminator(ParallelTaskTerminator((int) _max_worker_id, _task_queues)), - _has_overflown(false), _concurrent(false), _has_aborted(false), _restart_for_overflow(false), @@ -592,14 +591,9 @@ } -void G1ConcurrentMark::reset_marking_state(bool clear_overflow) { +void G1ConcurrentMark::reset_marking_state() { _global_mark_stack.set_should_expand(has_overflown()); _global_mark_stack.set_empty(); // Also clears the overflow stack's overflow flag - if (clear_overflow) { - clear_has_overflown(); - } else { - assert(has_overflown(), "pre-condition"); - } _finger = _heap_start; for (uint i = 0; i < _max_worker_id; ++i) { @@ -883,7 +877,7 @@ // not clear the overflow flag since we rely on it being true when // we exit this method to abort the pause and restart concurrent // marking. - reset_marking_state(true /* clear_overflow */); + reset_marking_state(); log_info(gc, marking)("Concurrent Mark reset for overflow"); } @@ -1752,12 +1746,6 @@ assert(_global_mark_stack.is_out_of_memory() || _global_mark_stack.is_empty(), "Mark stack should be empty (unless it is out of memory)"); - if (_global_mark_stack.is_out_of_memory()) { - // This should have been done already when we tried to push an - // entry on to the global mark stack. But let's do it again. - set_has_overflown(); - } - assert(rp->num_q() == active_workers, "why not"); rp->enqueue_discovered_references(executor); @@ -2931,7 +2919,6 @@ guarantee(_cm->mark_stack_empty(), "only way to reach here"); guarantee(_task_queue->size() == 0, "only way to reach here"); guarantee(!_cm->has_overflown(), "only way to reach here"); - guarantee(!_cm->mark_stack_overflow(), "only way to reach here"); } else { // Apparently there's more work to do. Let's abort this task. It // will restart it and we can hopefully find more things to do.