src/share/vm/gc_implementation/g1/concurrentMark.hpp

changeset 2973
23d434c6290d
parent 2969
6747fd0512e0
child 3065
ff53346271fe
     1.1 --- a/src/share/vm/gc_implementation/g1/concurrentMark.hpp	Mon Jun 20 09:42:26 2011 -0700
     1.2 +++ b/src/share/vm/gc_implementation/g1/concurrentMark.hpp	Mon Jun 20 22:03:13 2011 -0400
     1.3 @@ -605,10 +605,10 @@
     1.4    void mark_stack_pop(oop* arr, int max, int* n) {
     1.5      _markStack.par_pop_arr(arr, max, n);
     1.6    }
     1.7 -  size_t mark_stack_size()              { return _markStack.size(); }
     1.8 +  size_t mark_stack_size()                { return _markStack.size(); }
     1.9    size_t partial_mark_stack_size_target() { return _markStack.maxElems()/3; }
    1.10 -  bool mark_stack_overflow()            { return _markStack.overflow(); }
    1.11 -  bool mark_stack_empty()               { return _markStack.isEmpty(); }
    1.12 +  bool mark_stack_overflow()              { return _markStack.overflow(); }
    1.13 +  bool mark_stack_empty()                 { return _markStack.isEmpty(); }
    1.14  
    1.15    // (Lock-free) Manipulation of the region stack
    1.16    bool region_stack_push_lock_free(MemRegion mr) {
    1.17 @@ -833,8 +833,9 @@
    1.18      // _min_finger then we need to gray objects.
    1.19      // This routine is like registerCSetRegion but for an entire
    1.20      // collection of regions.
    1.21 -    if (max_finger > _min_finger)
    1.22 +    if (max_finger > _min_finger) {
    1.23        _should_gray_objects = true;
    1.24 +    }
    1.25    }
    1.26  
    1.27    // Returns "true" if at least one mark has been completed.
    1.28 @@ -880,14 +881,18 @@
    1.29    // The following indicate whether a given verbose level has been
    1.30    // set. Notice that anything above stats is conditional to
    1.31    // _MARKING_VERBOSE_ having been set to 1
    1.32 -  bool verbose_stats()
    1.33 -    { return _verbose_level >= stats_verbose; }
    1.34 -  bool verbose_low()
    1.35 -    { return _MARKING_VERBOSE_ && _verbose_level >= low_verbose; }
    1.36 -  bool verbose_medium()
    1.37 -    { return _MARKING_VERBOSE_ && _verbose_level >= medium_verbose; }
    1.38 -  bool verbose_high()
    1.39 -    { return _MARKING_VERBOSE_ && _verbose_level >= high_verbose; }
    1.40 +  bool verbose_stats() {
    1.41 +    return _verbose_level >= stats_verbose;
    1.42 +  }
    1.43 +  bool verbose_low() {
    1.44 +    return _MARKING_VERBOSE_ && _verbose_level >= low_verbose;
    1.45 +  }
    1.46 +  bool verbose_medium() {
    1.47 +    return _MARKING_VERBOSE_ && _verbose_level >= medium_verbose;
    1.48 +  }
    1.49 +  bool verbose_high() {
    1.50 +    return _MARKING_VERBOSE_ && _verbose_level >= high_verbose;
    1.51 +  }
    1.52  };
    1.53  
    1.54  // A class representing a marking task.
    1.55 @@ -1063,8 +1068,9 @@
    1.56    // respective limit and calls reached_limit() if they have
    1.57    void check_limits() {
    1.58      if (_words_scanned >= _words_scanned_limit ||
    1.59 -        _refs_reached >= _refs_reached_limit)
    1.60 +        _refs_reached >= _refs_reached_limit) {
    1.61        reached_limit();
    1.62 +    }
    1.63    }
    1.64    // this is supposed to be called regularly during a marking step as
    1.65    // it checks a bunch of conditions that might cause the marking step

mercurial