src/share/vm/gc_implementation/g1/g1RemSet.inline.hpp

changeset 1112
96b229c54d1e
parent 1061
87fa6e083d82
child 1279
bd02caa94611
child 1280
df6caf649ff7
     1.1 --- a/src/share/vm/gc_implementation/g1/g1RemSet.inline.hpp	Wed Mar 25 10:36:08 2009 -0400
     1.2 +++ b/src/share/vm/gc_implementation/g1/g1RemSet.inline.hpp	Wed Mar 25 13:10:54 2009 -0700
     1.3 @@ -65,7 +65,6 @@
     1.4    HeapRegion* to = _g1->heap_region_containing(obj);
     1.5    // The test below could be optimized by applying a bit op to to and from.
     1.6    if (to != NULL && from != NULL && from != to) {
     1.7 -    bool update_delayed = false;
     1.8      // There is a tricky infinite loop if we keep pushing
     1.9      // self forwarding pointers onto our _new_refs list.
    1.10      // The _par_traversal_in_progress flag is true during the collection pause,
    1.11 @@ -77,10 +76,7 @@
    1.12        // or processed (if an evacuation failure occurs) at the end
    1.13        // of the collection.
    1.14        // See HRInto_G1RemSet::cleanup_after_oops_into_collection_set_do().
    1.15 -      update_delayed = true;
    1.16 -    }
    1.17 -
    1.18 -    if (!to->popular() && !update_delayed) {
    1.19 +    } else {
    1.20  #if G1_REM_SET_LOGGING
    1.21        gclog_or_tty->print_cr("Adding " PTR_FORMAT " (" PTR_FORMAT ") to RS"
    1.22                               " for region [" PTR_FORMAT ", " PTR_FORMAT ")",
    1.23 @@ -88,9 +84,7 @@
    1.24                               to->bottom(), to->end());
    1.25  #endif
    1.26        assert(to->rem_set() != NULL, "Need per-region 'into' remsets.");
    1.27 -      if (to->rem_set()->add_reference(p, tid)) {
    1.28 -        _g1->schedule_popular_region_evac(to);
    1.29 -      }
    1.30 +      to->rem_set()->add_reference(p, tid);
    1.31      }
    1.32    }
    1.33  }

mercurial