# HG changeset patch # User ehelin # Date 1444301237 -7200 # Node ID 2083f82acec8424a9340f53313f29c0e4dbe07a1 # Parent 399fdb935d28c8b55afc4020cbc10c92d33ba99a 8139134: Wrong tenuring threshold in young GC trace event Reviewed-by: ehelin, jwilhelm Contributed-by: Carsten Varming diff -r 399fdb935d28 -r 2083f82acec8 hotspot/src/share/vm/gc/cms/parNewGeneration.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(); diff -r 399fdb935d28 -r 2083f82acec8 hotspot/src/share/vm/gc/parallel/psScavenge.cpp --- 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(); diff -r 399fdb935d28 -r 2083f82acec8 hotspot/src/share/vm/gc/serial/defNewGeneration.cpp --- 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();