8140251: Define the G1 term MMU somewhere in the source code.
authordrwhite
Wed, 21 Oct 2015 14:13:36 -0400
changeset 33598 e230df0ac751
parent 33597 5d42601232b1
child 33599 69416bbe5e84
8140251: Define the G1 term MMU somewhere in the source code. Summary: (MINOR) Simple comment fix to define Minimum Mutator Utilization (MMU) Reviewed-by: jwilhelm
hotspot/src/share/vm/gc/g1/g1MMUTracker.hpp
--- a/hotspot/src/share/vm/gc/g1/g1MMUTracker.hpp	Fri Oct 23 14:33:19 2015 +0300
+++ b/hotspot/src/share/vm/gc/g1/g1MMUTracker.hpp	Wed Oct 21 14:13:36 2015 -0400
@@ -29,11 +29,23 @@
 #include "memory/allocation.hpp"
 #include "utilities/debug.hpp"
 
-// Keeps track of the GC work and decides when it is OK to do GC work
+// Two major user controls over G1 behavior are setting a pause time goal (MaxGCPauseMillis),
+// over a time slice (GCPauseIntervalMillis). This defines the Minimum Mutator
+// Utilisation (MMU) goal.
+//
+// * Definitions *
+// Mutator Utilisation:
+// - for a given time slice duration "ts",
+// - mutator utilisation is the following fraction:
+//     non_gc_time / ts
+//
+// Minimum Mutator Utilisation (MMU):
+// - the worst mutator utilisation across all time slices.
+//
+// G1MMUTracker keeps track of the GC work and decides when it is OK to do GC work
 // and for how long so that the MMU invariants are maintained.
-
-/***** ALL TIMES ARE IN SECS!!!!!!! *****/
-
+//
+// ***** ALL TIMES ARE IN SECS!!!!!!! *****
 // this is the "interface"
 class G1MMUTracker: public CHeapObj<mtGC> {
 protected: