8139134: Wrong tenuring threshold in young GC trace event
Reviewed-by: ehelin, jwilhelm
Contributed-by: Carsten Varming <cvarming@twitter.com>
--- 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();