8065227: Report allocation context stats at end of cleanup

Fri, 21 Nov 2014 10:31:30 +0100

author
sjohanss
date
Fri, 21 Nov 2014 10:31:30 +0100
changeset 7370
8d27d6113625
parent 7369
b840813adfcc
child 7371
aa2442f89230

8065227: Report allocation context stats at end of cleanup
Summary: Moved allocation context update from remark to the cleanup phase.
Reviewed-by: mgerdin, jmasa

src/share/vm/gc_implementation/g1/concurrentMark.cpp file | annotate | diff | comparison | revisions
src/share/vm/gc_implementation/g1/g1AllocationContext.hpp file | annotate | diff | comparison | revisions
     1.1 --- a/src/share/vm/gc_implementation/g1/concurrentMark.cpp	Fri Nov 21 12:08:37 2014 +0100
     1.2 +++ b/src/share/vm/gc_implementation/g1/concurrentMark.cpp	Fri Nov 21 10:31:30 2014 +0100
     1.3 @@ -2175,6 +2175,7 @@
     1.4    // We reclaimed old regions so we should calculate the sizes to make
     1.5    // sure we update the old gen/space data.
     1.6    g1h->g1mm()->update_sizes();
     1.7 +  g1h->allocation_context_stats().update_after_mark();
     1.8  
     1.9    g1h->trace_heap_after_concurrent_cycle();
    1.10  }
    1.11 @@ -3346,7 +3347,6 @@
    1.12    } else {
    1.13      g1_par_agg_task.work(0);
    1.14    }
    1.15 -  _g1h->allocation_context_stats().update_at_remark();
    1.16  }
    1.17  
    1.18  // Clear the per-worker arrays used to store the per-region counting data
     2.1 --- a/src/share/vm/gc_implementation/g1/g1AllocationContext.hpp	Fri Nov 21 12:08:37 2014 +0100
     2.2 +++ b/src/share/vm/gc_implementation/g1/g1AllocationContext.hpp	Fri Nov 21 10:31:30 2014 +0100
     2.3 @@ -45,7 +45,7 @@
     2.4  public:
     2.5    inline void clear() { }
     2.6    inline void update(bool full_gc) { }
     2.7 -  inline void update_at_remark() { }
     2.8 +  inline void update_after_mark() { }
     2.9    inline bool available() { return false; }
    2.10  };
    2.11  

mercurial