RFR (XS): 8196883: G1RemSet::refine_card_concurrently doesn't need to check for cards in collection set
erik.helin at oracle.com
Thu Feb 8 08:27:45 UTC 2018
On 02/07/2018 05:25 PM, Thomas Schatzl wrote:
> Hi all,
> can I have reviews for this small change that removes obsolete code
> in concurrent refinement processing?
> from the CR: "G1RemSet::refine_card_concurrently tests whether the
> region is in the collection set, skipping refinement if so, with an
> explanation that objects in the cset are about to move. But the cset is
> a collection pause artifact, and is clear when not in a GC pause. The
> test was useful when there was a single refine_card function that was
> used both concurrently and during a GC pause, but JDK-8177707 split the
> two cases into separate functions. It is no longer useful, and should
> be removed."
Looks good, Reviewed!
> hs-tier 1-5
More information about the hotspot-gc-dev