RFR: Add generations to freeset [v8]

Kelvin Nilsen kdnilsen at openjdk.org
Thu Apr 20 19:43:52 UTC 2023


On Thu, 20 Apr 2023 19:05:45 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp line 125:
>> 
>>> 123: 
>>> 124: 
>>> 125: HeapWord* ShenandoahFreeSet::allocate_old_with_affiliation(ShenandoahAffiliation affiliation,
>> 
>> Will `affiliation` always be `old` here? Should we remove this parameter?
>
> Affiliation might be FREE.  First, we try to allocate from within an old-collector-region that is_old().  If that fails, we'll try to allocate from an old_collector_region that is not afilliated.

I'm adding a comment to explain this.

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/250#discussion_r1173017071


More information about the shenandoah-dev mailing list