RFR: Pacer for evacuation should print "Avail" to capture discounting

Aleksey Shipilev shade at redhat.com
Thu Jul 12 16:34:45 UTC 2018


Pacer for Evacuation prints "Free" for the value that is actually free minus cset, because it
discounts cset for future evacs. Makes the log misleading, better to say "Avail":

diff -r 2ce3f06bf1ff src/hotspot/share/gc/shenandoah/shenandoahPacer.cpp
--- a/src/hotspot/share/gc/shenandoah/shenandoahPacer.cpp       Thu Jul 12 12:19:26 2018 +0200
+++ b/src/hotspot/share/gc/shenandoah/shenandoahPacer.cpp       Thu Jul 12 12:20:42 2018 +0200
@@ -100,7 +100,7 @@

   restart_with(non_taxable, tax);

-  log_info(gc, ergo)("Pacer for Evacuation. Used CSet: " SIZE_FORMAT "M, Free: " SIZE_FORMAT
+  log_info(gc, ergo)("Pacer for Evacuation. Used CSet: " SIZE_FORMAT "M, Avail: " SIZE_FORMAT
                      "M, Non-Taxable: " SIZE_FORMAT "M, Alloc Tax Rate: %.1fx",
                      used / M, free / M, non_taxable / M, tax);
 }

Testing: eyeballing logs

-Aleksey



More information about the shenandoah-dev mailing list