RFR (M) JDK-8151178: Move the collection set out of the G1 collector policy
jesper.wilhelmsson at oracle.com
Fri Mar 4 14:57:10 UTC 2016
From a purely aesthetic point of view I would appreciate if you cleaned up
lines 78, 145, 156 in g1CollectionSet.cpp
Also, most files could use a new copyright date if you care about that sort of
Den 4/3/16 kl. 09:35, skrev Mikael Gerdin:
> Hi all,
> As part of an attempt to make it easier for G1 to contain multiple collector
> policies I suggest that the collection set code should be moved out of the
> current G1 collector policy class into a class of its own.
> I've tried to hold back on doing further cleanups in this patch in order to make
> it easier to review the changes.
> I do have some ideas on how the collection set code could be further cleaned up
> but I'm not sure if I will have time to test them properly before FC.
> I've also split the change into three webrevs in the version1 directory:
> webrev_1 consists of moving the code and changing code accessing collection set
> related methods
> webrev_2 consists of renaming almost all methods in the new class to not contain
> redundant collection set naming
> webrev_3 restores the functionality for collector policy extensions to hook into
> collection set selection.
> Testing: JPRT, RBT GC testing.
More information about the hotspot-gc-dev