changeset 38107:aee6db99b6c7

8155087: Calculation in other_time_ms() is incorrect Reviewed-by: tschatzl, ehelin
author sjohanss
date Wed, 27 Apr 2016 16:02:32 +0200
parents da14ba2f0cd9
children 95c7e9d6747c
files hotspot/src/share/vm/gc/g1/g1DefaultPolicy.cpp
diffstat 1 files changed, 1 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/hotspot/src/share/vm/gc/g1/g1DefaultPolicy.cpp	Wed Apr 27 16:02:02 2016 +0200
+++ b/hotspot/src/share/vm/gc/g1/g1DefaultPolicy.cpp	Wed Apr 27 16:02:32 2016 +0200
@@ -549,11 +549,7 @@
 }
 
 double G1DefaultPolicy::other_time_ms(double pause_time_ms) const {
-  return pause_time_ms -
-         average_time_ms(G1GCPhaseTimes::UpdateRS) -
-         average_time_ms(G1GCPhaseTimes::ScanRS) -
-         average_time_ms(G1GCPhaseTimes::ObjCopy) -
-         average_time_ms(G1GCPhaseTimes::Termination);
+  return pause_time_ms - phase_times()->cur_collection_par_time_ms();
 }
 
 double G1DefaultPolicy::constant_other_time_ms(double pause_time_ms) const {