RFR: 8144040: UpdateRSetDeferred in G1EvacFailure will never visit survivor regions
mikael.gerdin at oracle.com
Thu Nov 26 12:50:17 UTC 2015
On 2015-11-25 18:37, Erik Helin wrote:
> On 2015-11-25, Thomas Schatzl wrote:
>> On Wed, 2015-11-25 at 15:39 +0100, Erik Helin wrote:
>>> Hi all,
>>> this patch removes an unneeded check from UpdateRSetDeferred which is used in
>>> the case of an evacuation failure. When we start a GC with G1, we
>>> change the type of all survivor regions to eden (see the call to
>>> hr->set_eden_pre_gc() in G1CollectorPolicy::finalize_young_cset_part).
>>> Therefore, if we ever get an evac failure in a region, then that region
>>> must be either young or old. Therefore, we don't need to check for
>>> survivor regions in UpdateRSetDeferred.
>>> - GC test suite with full verification
>> looks good.
More information about the hotspot-gc-dev