Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8155087: Calculation in other_time_ms() is incorrect
authorsjohanss
Wed, 27 Apr 2016 16:02:32 +0200
changeset 38107 aee6db99b6c7
parent 38106 da14ba2f0cd9
child 38108 95c7e9d6747c
8155087: Calculation in other_time_ms() is incorrect Reviewed-by: tschatzl, ehelin
hotspot/src/share/vm/gc/g1/g1DefaultPolicy.cpp file | annotate | diff | comparison | revisions 
--- 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 {
jdk-sandbox