8139134: Wrong tenuring threshold in young GC trace event
authorehelin
Thu, 08 Oct 2015 12:47:17 +0200
changeset 33143 2083f82acec8
parent 33142 399fdb935d28
child 33144 c56850e328fc
8139134: Wrong tenuring threshold in young GC trace event Reviewed-by: ehelin, jwilhelm Contributed-by: Carsten Varming <cvarming@twitter.com>
hotspot/src/share/vm/gc/cms/parNewGeneration.cpp
hotspot/src/share/vm/gc/parallel/psScavenge.cpp
hotspot/src/share/vm/gc/serial/defNewGeneration.cpp
--- a/hotspot/src/share/vm/gc/cms/parNewGeneration.cpp	Fri Oct 02 11:46:42 2015 +0200
+++ b/hotspot/src/share/vm/gc/cms/parNewGeneration.cpp	Thu Oct 08 12:47:17 2015 +0200
@@ -968,6 +968,8 @@
                                               _gc_timer);
   }
   _gc_tracer.report_gc_reference_stats(stats);
+  _gc_tracer.report_tenuring_threshold(tenuring_threshold());
+
   if (!promotion_failed()) {
     // Swap the survivor spaces.
     eden()->clear(SpaceDecorator::Mangle);
@@ -1030,7 +1032,6 @@
   rp->verify_no_references_recorded();
 
   gch->trace_heap_after_gc(gc_tracer());
-  _gc_tracer.report_tenuring_threshold(tenuring_threshold());
 
   _gc_timer->register_gc_end();
 
--- a/hotspot/src/share/vm/gc/parallel/psScavenge.cpp	Fri Oct 02 11:46:42 2015 +0200
+++ b/hotspot/src/share/vm/gc/parallel/psScavenge.cpp	Thu Oct 08 12:47:17 2015 +0200
@@ -475,6 +475,8 @@
       }
     }
 
+    _gc_tracer.report_tenuring_threshold(tenuring_threshold());
+
     // Let the size policy know we're done.  Note that we count promotion
     // failure cleanup time as part of the collection (otherwise, we're
     // implicitly saying it's mutator time).
@@ -674,7 +676,6 @@
 
   heap->print_heap_after_gc();
   heap->trace_heap_after_gc(&_gc_tracer);
-  _gc_tracer.report_tenuring_threshold(tenuring_threshold());
 
   if (ZapUnusedHeapArea) {
     young_gen->eden_space()->check_mangled_unused_area_complete();
--- a/hotspot/src/share/vm/gc/serial/defNewGeneration.cpp	Fri Oct 02 11:46:42 2015 +0200
+++ b/hotspot/src/share/vm/gc/serial/defNewGeneration.cpp	Thu Oct 08 12:47:17 2015 +0200
@@ -648,6 +648,7 @@
   rp->process_discovered_references(&is_alive, &keep_alive, &evacuate_followers,
                                     NULL, _gc_timer);
   gc_tracer.report_gc_reference_stats(stats);
+  gc_tracer.report_tenuring_threshold(tenuring_threshold());
 
   if (!_promotion_failed) {
     // Swap the survivor spaces.
@@ -712,7 +713,6 @@
   update_time_of_last_gc(now);
 
   gch->trace_heap_after_gc(&gc_tracer);
-  gc_tracer.report_tenuring_threshold(tenuring_threshold());
 
   _gc_timer->register_gc_end();