RFR (XS): 8183397: Ensure consistent closure filtering during evacuation
thomas.schatzl at oracle.com
Mon Jul 3 11:24:48 UTC 2017
can I have reviews for this change that fixes an observation that has
been made recently by Erik, i.e. that the "else" part of several
evacuation closures inconsistently filters out non-cross-region
references before checking whether the referenced object is a humongous
or ext region.
This causes somewhat hard to diagnose performance issues, and earlier
filtering does not hurt if done anyway.
(Note that the current way of checking in all but the UpdateRS closure
using HeapRegion::is_in_same_region() seems optimal. The only reason
why the other way in the UpdateRS closure is better because the code
needs the "to" HeapRegion pointer anyway)
jprt, performance regression analysis
More information about the hotspot-gc-dev