RFR: Change affiliation representation [v2]

Kelvin Nilsen kdnilsen at openjdk.org
Wed Nov 16 14:46:33 UTC 2022


On Mon, 14 Nov 2022 21:53:49 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp line 356:
>> 
>>> 354: 
>>> 355:   _regions = NEW_C_HEAP_ARRAY(ShenandoahHeapRegion*, _num_regions, mtGC);
>>> 356:   _affiliations = NEW_C_HEAP_ARRAY(uint8_t, _num_regions, mtGC);
>> 
>> Should we only initialize this for generational mode? or are there calls to check affiliation in the other modes too?
>
> Good point.  I think it's only needed for generational mode.

After further investigation, I see that we need _affiliation for non-generational mode to keep track of FREE status.  I've added a comment to this effect.

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

PR: https://git.openjdk.org/shenandoah/pull/170


More information about the shenandoah-dev mailing list