archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-08-27 06:45:07 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

6888316: G1: has_aborted() || _cm->region_stack_empty() fails

Browse source 
Remove incorrect guarantee.

Reviewed-by: apetrusenko, iveresov
This commit is contained in:
Antonios Printezis 2009-10-07 09:42:18 -04:00
parent 31de6adee9
commit 0835f0c4df
1 changed files with 0 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

7
hotspot/src/share/vm/gc_implementation/g1/concurrentMark.cpp
View file

@ -3416,13 +3416,6 @@ void CMTask::drain_region_stack(BitMapClosure* bc) {
_region_finger = NULL; _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()) if (_cm->verbose_low())
gclog_or_tty->print_cr("[%d] drained region stack, size = %d", gclog_or_tty->print_cr("[%d] drained region stack, size = %d",
_task_id, _cm->region_stack_size()); _task_id, _cm->region_stack_size());