RFR: 8277736: G1: Allow forced evacuation failure of first N regions in collection set

Albert Mingkun Yang ayang at openjdk.java.net
Tue Nov 30 10:02:05 UTC 2021


On Fri, 26 Nov 2021 02:34:41 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> src/hotspot/share/gc/g1/g1YoungGCEvacFailureInjector.cpp line 47:
>> 
>>> 45:       _regions.set_bit(r->hrm_index());
>>> 46:       --_evac_failure_regions_num;
>>> 47:       return false;
>> 
>> Just an initial comment to think about and discuss: This would most likely always ever select the same regions (probably eden) that are first in the collection set.
>> I think selecting by probability (with G1EvacuationFailureALotCSetPercent uniform probability) would be more interesting even if it makes the selection more random.
>
> Yes, currently I only make it select regions starting from first.
> The reason I don't add random selection is that, currently we need this functionality to verify the functionality and perf improvement of JDK-8256265 implementation, so a more stable selection of evacuation failure regions will help to verify the effect of JDK-8256265.
> 
> I agree that random selection is an interesting option to supply too, I just file https://bugs.openjdk.java.net/browse/JDK-8277851 to track the issue.

This PR makes the first X% regions in cset evac-fail, but `G1EvacuationFailureALotCSetPercent` doesn't show such bias. (It kind of implies uniform distribution, IMO.) It would be nice if the name and/or the explanation string could be more specific.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6561


More information about the hotspot-gc-dev mailing list