Mercurial > hg > openjdk > hsx20

changeset 1021:4c3458a31e17

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
6888316: G1: has_aborted() || _cm->region_stack_empty() fails Summary: Remove incorrect guarantee. Reviewed-by: apetrusenko, iveresov
author tonyp
date Wed, 07 Oct 2009 09:42:18 -0400
parents 1f19207eefc2
children 11d4857fe5e1
files src/share/vm/gc_implementation/g1/concurrentMark.cpp
diffstat 1 files changed, 0 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/gc_implementation/g1/concurrentMark.cpp	Mon Oct 05 12:05:48 2009 -0400
+++ b/src/share/vm/gc_implementation/g1/concurrentMark.cpp	Wed Oct 07 09:42:18 2009 -0400
@@ -3416,13 +3416,6 @@
       _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());