src/share/vm/gc/g1/g1ConcurrentMark.inline.hpp

rev 13284 : imported patch 8184346-cleanup-g1cmbitmap
rev 13286 : imported patch 8184346-erikd-mgerdin-review
rev 13287 : [mq]: 8184346-erikd-review
rev 13288 : [mq]: 8184347-move-g1cmbitmap-into-own-files

*** 25,94 ****
  #ifndef SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP
  #define SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP
  
  #include "gc/g1/g1CollectedHeap.inline.hpp"
  #include "gc/g1/g1ConcurrentMark.hpp"
  #include "gc/g1/g1ConcurrentMarkObjArrayProcessor.inline.hpp"
  #include "gc/g1/suspendibleThreadSet.hpp"
  #include "gc/shared/taskqueue.inline.hpp"
  #include "utilities/bitMap.inline.hpp"
  
  inline bool G1ConcurrentMark::par_mark(oop obj) {
    return _nextMarkBitMap->par_mark((HeapWord*)obj);
  }
  
- inline bool G1CMBitMap::iterate(G1CMBitMapClosure* cl, MemRegion mr) {
-   assert(!mr.is_empty(), "Does not support empty memregion to iterate over");
-   assert(_covered.contains(mr), "Given MemRegion from " PTR_FORMAT " to " PTR_FORMAT " not contained in heap area", p2i(mr.start()), p2i(mr.end()));
- 
-   BitMap::idx_t const end_offset = addr_to_offset(mr.end());
-   BitMap::idx_t offset = _bm.get_next_one_offset(addr_to_offset(mr.start()), end_offset);
- 
-   while (offset < end_offset) {
-     HeapWord* const addr = offset_to_addr(offset);
-     if (!cl->do_addr(addr)) {
-       return false;
-     }
-     size_t const obj_size = (size_t)((oop)addr)->size();
-     offset = _bm.get_next_one_offset(offset + (obj_size >> _shifter), end_offset);
-   }
-   return true;
- }
- 
- inline HeapWord* G1CMBitMap::get_next_marked_addr(const HeapWord* addr,
-                                                   const HeapWord* limit) const {
-   assert(limit != NULL, "limit must not be NULL");
-   // Round addr up to a possible object boundary to be safe.
-   size_t const addr_offset = addr_to_offset(align_up(addr, HeapWordSize << _shifter));
-   size_t const limit_offset = addr_to_offset(limit);
-   size_t const nextOffset = _bm.get_next_one_offset(addr_offset, limit_offset);
-   return offset_to_addr(nextOffset);
- }
- 
- #ifdef ASSERT
- inline void G1CMBitMap::check_mark(HeapWord* addr) {
-   assert(G1CollectedHeap::heap()->is_in_exact(addr),
-          "Trying to access bitmap " PTR_FORMAT " for address " PTR_FORMAT " not in the heap.",
-          p2i(this), p2i(addr));
- }
- #endif
- 
- inline void G1CMBitMap::mark(HeapWord* addr) {
-   check_mark(addr);
-   _bm.set_bit(addr_to_offset(addr));
- }
- 
- inline void G1CMBitMap::clear(HeapWord* addr) {
-   check_mark(addr);
-   _bm.clear_bit(addr_to_offset(addr));
- }
- 
- inline bool G1CMBitMap::par_mark(HeapWord* addr) {
-   check_mark(addr);
-   return _bm.par_set_bit(addr_to_offset(addr));
- }
- 
  #ifndef PRODUCT
  template<typename Fn>
  inline void G1CMMarkStack::iterate(Fn fn) const {
    assert_at_safepoint(true);
  
--- 25,44 ----
  #ifndef SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP
  #define SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP
  
  #include "gc/g1/g1CollectedHeap.inline.hpp"
  #include "gc/g1/g1ConcurrentMark.hpp"
+ #include "gc/g1/g1ConcurrentMarkBitMap.inline.hpp"
  #include "gc/g1/g1ConcurrentMarkObjArrayProcessor.inline.hpp"
  #include "gc/g1/suspendibleThreadSet.hpp"
  #include "gc/shared/taskqueue.inline.hpp"
  #include "utilities/bitMap.inline.hpp"
  
  inline bool G1ConcurrentMark::par_mark(oop obj) {
    return _nextMarkBitMap->par_mark((HeapWord*)obj);
  }
  
  #ifndef PRODUCT
  template<typename Fn>
  inline void G1CMMarkStack::iterate(Fn fn) const {
    assert_at_safepoint(true);