8139134: Wrong tenuring threshold in young GC trace event

Reviewed-by: ehelin, jwilhelm
This commit is contained in:
Carsten Varming 2015-10-08 12:47:17 +02:00 committed by Erik Helin
parent 56d0ec1e1a
commit 3f29249aee
3 changed files with 5 additions and 3 deletions
hotspot/src/share/vm/gc

@ -968,6 +968,8 @@ void ParNewGeneration::collect(bool full,
_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 @@ void ParNewGeneration::collect(bool full,
rp->verify_no_references_recorded();
gch->trace_heap_after_gc(gc_tracer());
_gc_tracer.report_tenuring_threshold(tenuring_threshold());
_gc_timer->register_gc_end();

@ -475,6 +475,8 @@ bool PSScavenge::invoke_no_policy() {
}
}
_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 @@ bool PSScavenge::invoke_no_policy() {
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();

@ -648,6 +648,7 @@ void DefNewGeneration::collect(bool full,
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 @@ void DefNewGeneration::collect(bool full,
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();