Fri, 21 Nov 2014 10:31:30 +0100
8065227: Report allocation context stats at end of cleanup
Summary: Moved allocation context update from remark to the cleanup phase.
Reviewed-by: mgerdin, jmasa
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