changeset 9432:371ac7d4ccb2

8139134: Wrong tenuring threshold in young GC trace event Reviewed-by: ehelin, jwilhelm Contributed-by: Carsten Varming <cvarming@twitter.com>
author ehelin
date Thu, 08 Oct 2015 12:47:17 +0200
parents 17986acb4825
children 5459f44b1a75
files src/share/vm/gc/cms/parNewGeneration.cpp src/share/vm/gc/parallel/psScavenge.cpp src/share/vm/gc/serial/defNewGeneration.cpp
diffstat 3 files changed, 5 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/gc/cms/parNewGeneration.cpp	Fri Oct 02 11:46:42 2015 +0200
+++ b/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/src/share/vm/gc/parallel/psScavenge.cpp	Fri Oct 02 11:46:42 2015 +0200
+++ b/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/src/share/vm/gc/serial/defNewGeneration.cpp	Fri Oct 02 11:46:42 2015 +0200
+++ b/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();