6861557: G1: assert(top() == bottom() || zfs == Allocated,"Region must be empty, or ...")

Tue, 15 Sep 2009 16:50:50 -0700

author
ysr
date
Tue, 15 Sep 2009 16:50:50 -0700
changeset 1395
eb058abd0562
parent 1380
68ef3fdcdb76
child 1396
d7c9544cc141

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

src/share/vm/gc_implementation/g1/heapRegionSeq.cpp file | annotate | diff | comparison | revisions
     1.1 --- a/src/share/vm/gc_implementation/g1/heapRegionSeq.cpp	Thu Sep 10 16:46:17 2009 -0700
     1.2 +++ b/src/share/vm/gc_implementation/g1/heapRegionSeq.cpp	Tue Sep 15 16:50:50 2009 -0700
     1.3 @@ -302,9 +302,9 @@
     1.4      if (cur->isHumongous()) {
     1.5        return MemRegion(last_start, end);
     1.6      }
     1.7 -    cur->reset_zero_fill();
     1.8      assert(cur == _regions.top(), "Should be top");
     1.9      if (!cur->is_empty()) break;
    1.10 +    cur->reset_zero_fill();
    1.11      shrink_bytes -= cur->capacity();
    1.12      num_regions_deleted++;
    1.13      _regions.pop();

mercurial