RFR (M): 8136681: Factor out IHOP calculation from G1CollectorPolicy
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Nov 5 09:42:11 UTC 2015
Hi all,
can I have reviews for this preparatory change for adaptive IHOP
sizing, that factors out the IHOP calculation from G1Collectorpolicy
into separate classes?
So instead of having this single line at G1CollectorPolicy:944
944 size_t marking_initiating_used_threshold =
945 (_g1->capacity() / 100) * InitiatingHeapOccupancyPercent;
[...]
949 if ((cur_used_bytes + alloc_byte_size) >
marking_initiating_used_threshold) {
that determines whether marking should start, that decision is
externalized into a G1IHOPControl class with a
get_conc_mark_start_threshold() method.
This allows flexible replacement of the IHOP calculation algorithm
depending on some switch using a factory method.
In addition to that, this change tracks and passes several information
IHOPControl instances need, like allocation between gcs, mutator time
from initial mark to the first mixed gc, and others.
The change also adds some test.
It depends on all my recent patches in this area, namely:
8140689: Skip last young-only gc if nothing to do in the mixed gc phase
8140597: Forcing an initial mark causes G1 to abort mixed collections
8140585: PLAB statistics are flushed too late
8139874: After G1 Full GC, the next GC is always a young-only GC
8138740: Start initial mark right after mixed GC if needed
The problem is that it expects (and verifies) that the GC pauses are
executed in the "correct" order.
There is no behavioral change in this CR.
CR:
https://bugs.openjdk.java.net/browse/JDK-8136681
Webrev:
http://cr.openjdk.java.net/~tschatzl/8136681/webrev/
Testing:
jprt, vm.gc, lots of manual testing, new VM test
Note that SQE will create additional tests.
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list