RFR: Move affiliation and mode out of ShenandoahGenerationalMode [v2]

William Kemper wkemper at openjdk.org
Thu Apr 6 16:37:58 UTC 2023


On Thu, 6 Apr 2023 16:14:38 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix Zero builds
>
> src/hotspot/share/gc/shenandoah/shenandoahOldGeneration.cpp line 185:
> 
>> 183: bool ShenandoahOldGeneration::contains(ShenandoahHeapRegion* region) const {
>> 184:   // TODO: Should this be region->is_old() instead?
>> 185:   return !region->is_young();
> 
> Agree with proposed change.  How could this not be a problem?   Is it used?

We have to be careful with this one because some uses expect to visit regions that are both old _and_ free (or young _and_ free). Maybe we should get rid of this method and be explicit about it?

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/246#discussion_r1160026291


More information about the shenandoah-dev mailing list