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

rev 6588 : 8047821: G1 Does not use the save_marks functionality as intended
Reviewed-by:

*** 4370,4380 ****
    if (retained_region != NULL &&
        !retained_region->in_collection_set() &&
        !(retained_region->top() == retained_region->end()) &&
        !retained_region->is_empty() &&
        !retained_region->isHumongous()) {
!     retained_region->set_saved_mark();
      // The retained region was added to the old region set when it was
      // retired. We have to remove it now, since we don't allow regions
      // we allocate to in the region sets. We'll re-add it later, when
      // it's retired again.
      _old_set.remove(retained_region);
--- 4370,4380 ----
    if (retained_region != NULL &&
        !retained_region->in_collection_set() &&
        !(retained_region->top() == retained_region->end()) &&
        !retained_region->is_empty() &&
        !retained_region->isHumongous()) {
!     retained_region->record_top_and_timestamp();
      // The retained region was added to the old region set when it was
      // retired. We have to remove it now, since we don't allow regions
      // we allocate to in the region sets. We'll re-add it later, when
      // it's retired again.
      _old_set.remove(retained_region);
*** 6619,6629 ****
                                                true /* do_expand */);
      if (new_alloc_region != NULL) {
        // We really only need to do this for old regions given that we
        // should never scan survivors. But it doesn't hurt to do it
        // for survivors too.
!       new_alloc_region->set_saved_mark();
        if (survivor) {
          new_alloc_region->set_survivor();
          _hr_printer.alloc(new_alloc_region, G1HRPrinter::Survivor);
          check_bitmaps("Survivor Region Allocation", new_alloc_region);
        } else {
--- 6619,6629 ----
                                                true /* do_expand */);
      if (new_alloc_region != NULL) {
        // We really only need to do this for old regions given that we
        // should never scan survivors. But it doesn't hurt to do it
        // for survivors too.
!       new_alloc_region->record_top_and_timestamp();
        if (survivor) {
          new_alloc_region->set_survivor();
          _hr_printer.alloc(new_alloc_region, G1HRPrinter::Survivor);
          check_bitmaps("Survivor Region Allocation", new_alloc_region);
        } else {