From e7b7dca9a2951a39be696d38f659b3e844d9a74e Mon Sep 17 00:00:00 2001 From: tonyp Date: Wed, 7 Oct 2009 09:42:18 -0400 Subject: [PATCH] 6888316: G1: has_aborted() || _cm->region_stack_empty() fails Summary: Remove incorrect guarantee. Reviewed-by: apetrusenko, iveresov --- src/share/vm/gc_implementation/g1/concurrentMark.cpp | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/share/vm/gc_implementation/g1/concurrentMark.cpp b/src/share/vm/gc_implementation/g1/concurrentMark.cpp index 59ba0f59c..2ed4c6700 100644 --- a/src/share/vm/gc_implementation/g1/concurrentMark.cpp +++ b/src/share/vm/gc_implementation/g1/concurrentMark.cpp @@ -3416,13 +3416,6 @@ void CMTask::drain_region_stack(BitMapClosure* bc) { _region_finger = NULL; } - // We only push regions on the region stack during evacuation - // pauses. So if we come out the above iteration because we region - // stack is empty, it will remain empty until the next yield - // point. So, the guarantee below is safe. - guarantee( has_aborted() || _cm->region_stack_empty(), - "only way to exit the loop" ); - if (_cm->verbose_low()) gclog_or_tty->print_cr("[%d] drained region stack, size = %d", _task_id, _cm->region_stack_size()); -- GitLab