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

Hamlin Li mli at openjdk.java.net
Wed Dec 1 11:23:29 UTC 2021


On Wed, 1 Dec 2021 11:02:35 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Albert review
>
> src/hotspot/share/gc/g1/g1_globals.hpp line 79:
> 
>> 77:                                                                             \
>> 78:   product(uintx, G1EvacuationFailureALotCSetPercent, 100,                   \
>> 79:           "Percent of the regions in cset that will fail evacuation")       \
> 
> Please spell out "cset" in the description text though as it is done everywhere else in `g1_globals.hpp`. Considering Albert's suggestion, maybe something like:
> 
> 
> The percentage of regions in the collection set starting from the beginning where the forced evacuation failure injection will be applied.

Sure. Thanks!

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

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



More information about the hotspot-gc-dev mailing list