Integrated: 8364760: G1: Remove obsolete code in G1MergeCardSetClosure

Thomas Schatzl tschatzl at openjdk.org
Fri Aug 8 08:00:18 UTC 2025


On Tue, 5 Aug 2025 16:12:32 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review this removal of some obsolete code.
> 
> Before JDK-8343782 G1 iterated remsets for humongous eager reclaim candidates by region, as there has been a 1:1 mapping between regions and (card-based) remembered sets.
> 
> With that change, all remembered set iteration should go via the group remsets (`G1CSetCandidateGroup`), and the code does, except for humongous eager reclaim candidates which handle their group remsets on the side (which is a separate issue).
> 
> This obsoleted some code in `G1MergeCardSetClosure` - it does not need to be a `G1HeapRegionClosure` any more, and for humongous regions we can directly do the iteration via the group rem set attached to the humongous starts region. I.e. inline that `merge_card_set_for_region()` method.
> 
> Testing: gha
> 
> Thanks,
>   Thomas

This pull request has now been integrated.

Changeset: 47017e38
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/47017e38642a58fd6425ec68c1fed96f19f39404
Stats:     38 lines in 1 file changed: 0 ins; 35 del; 3 mod

8364760: G1: Remove obsolete code in G1MergeCardSetClosure

Reviewed-by: ayang, sangheki

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

PR: https://git.openjdk.org/jdk/pull/26645


More information about the hotspot-gc-dev mailing list