src/share/vm/memory/space.cpp

rev 5732 : [mq]: comments2

*** 110,120 ****
  
    // Some collectors need to do special things whenever their dirty
    // cards are processed. For instance, CMS must remember mutator updates
    // (i.e. dirty cards) so as to re-scan mutated objects.
    // Such work can be piggy-backed here on dirty card scanning, so as to make
!   // it slightly more efficient than doing a complete non-detructive pre-scan
    // of the card table.
    MemRegionClosure* pCl = _sp->preconsumptionDirtyCardClosure();
    if (pCl != NULL) {
      pCl->do_MemRegion(mr);
    }
--- 110,120 ----
  
    // Some collectors need to do special things whenever their dirty
    // cards are processed. For instance, CMS must remember mutator updates
    // (i.e. dirty cards) so as to re-scan mutated objects.
    // Such work can be piggy-backed here on dirty card scanning, so as to make
!   // it slightly more efficient than doing a complete non-destructive pre-scan
    // of the card table.
    MemRegionClosure* pCl = _sp->preconsumptionDirtyCardClosure();
    if (pCl != NULL) {
      pCl->do_MemRegion(mr);
    }
*** 322,333 ****
    Space::set_bottom(new_bottom);
    _offsets.set_bottom(new_bottom);
  }
  
  void OffsetTableContigSpace::set_end(HeapWord* new_end) {
!   // Space should not advertize an increase in size
!   // until after the underlying offest table has been enlarged.
    _offsets.resize(pointer_delta(new_end, bottom()));
    Space::set_end(new_end);
  }
  
  #ifndef PRODUCT
--- 322,333 ----
    Space::set_bottom(new_bottom);
    _offsets.set_bottom(new_bottom);
  }
  
  void OffsetTableContigSpace::set_end(HeapWord* new_end) {
!   // Space should not advertise an increase in size
!   // until after the underlying offset table has been enlarged.
    _offsets.resize(pointer_delta(new_end, bottom()));
    Space::set_end(new_end);
  }
  
  #ifndef PRODUCT
*** 727,737 ****
    if (is_empty()) return;
    WaterMark bm = bottom_mark();
    object_iterate_from(bm, blk);
  }
  
! // For a continguous space object_iterate() and safe_object_iterate()
  // are the same.
  void ContiguousSpace::safe_object_iterate(ObjectClosure* blk) {
    object_iterate(blk);
  }
  
--- 727,737 ----
    if (is_empty()) return;
    WaterMark bm = bottom_mark();
    object_iterate_from(bm, blk);
  }
  
! // For a ContiguousSpace object_iterate() and safe_object_iterate()
  // are the same.
  void ContiguousSpace::safe_object_iterate(ObjectClosure* blk) {
    object_iterate(blk);
  }