6861557: G1: assert(top() == bottom() || zfs == Allocated,"Region must be empty, or ...")
Summary: Druing heap shrinking, check for emptiness of a region before modifying its ZF status.
Reviewed-by: tonyp
--- a/src/share/vm/gc_implementation/g1/heapRegionSeq.cpp Thu Sep 10 16:46:17 2009 -0700
+++ b/src/share/vm/gc_implementation/g1/heapRegionSeq.cpp Tue Sep 15 16:50:50 2009 -0700
@@ -302,9 +302,9 @@ MemRegion HeapRegionSeq::shrink_by(size_
if (cur->isHumongous()) {
return MemRegion(last_start, end);
}
- cur->reset_zero_fill();
assert(cur == _regions.top(), "Should be top");
if (!cur->is_empty()) break;
+ cur->reset_zero_fill();
shrink_bytes -= cur->capacity();
num_regions_deleted++;
_regions.pop();