src/share/vm/gc_implementation/g1/heapRegion.cpp

rev 5732 : [mq]: comments2

*** 1025,1035 ****
          *failures = true;
          return;
        }
      }
  
!     // Loook up end - 1
      HeapWord* addr_4 = the_end - 1;
      HeapWord* b_start_4 = _offsets.block_start_const(addr_4);
      if (b_start_4 != p) {
        gclog_or_tty->print_cr("BOT look up for end - 1: "PTR_FORMAT" "
                               " yielded "PTR_FORMAT", expecting "PTR_FORMAT,
--- 1025,1035 ----
          *failures = true;
          return;
        }
      }
  
!     // Look up end - 1
      HeapWord* addr_4 = the_end - 1;
      HeapWord* b_start_4 = _offsets.block_start_const(addr_4);
      if (b_start_4 != p) {
        gclog_or_tty->print_cr("BOT look up for end - 1: "PTR_FORMAT" "
                               " yielded "PTR_FORMAT", expecting "PTR_FORMAT,
*** 1109,1119 ****
      // about to start scanning this region. If it does so after
      // set_saved_mark and before _gc_time_stamp = ..., then the latter
      // will be false, and it will pick up top() as the high water mark
      // of region. If it does so after _gc_time_stamp = ..., then it
      // will pick up the right saved_mark_word() as the high water mark
!     // of the region. Either way, the behaviour will be correct.
      ContiguousSpace::set_saved_mark();
      OrderAccess::storestore();
      _gc_time_stamp = curr_gc_time_stamp;
      // No need to do another barrier to flush the writes above. If
      // this is called in parallel with other threads trying to
--- 1109,1119 ----
      // about to start scanning this region. If it does so after
      // set_saved_mark and before _gc_time_stamp = ..., then the latter
      // will be false, and it will pick up top() as the high water mark
      // of region. If it does so after _gc_time_stamp = ..., then it
      // will pick up the right saved_mark_word() as the high water mark
!     // of the region. Either way, the behavior will be correct.
      ContiguousSpace::set_saved_mark();
      OrderAccess::storestore();
      _gc_time_stamp = curr_gc_time_stamp;
      // No need to do another barrier to flush the writes above. If
      // this is called in parallel with other threads trying to