RFR: 8324649: Shenandoah: refactor implementation of free set [v6]

Kelvin Nilsen kdnilsen at openjdk.org
Sat Feb 3 16:10:04 UTC 2024


On Sat, 3 Feb 2024 15:46:04 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp line 761:
>> 
>>> 759: void ShenandoahFreeSet::prepare_to_rebuild(size_t &cset_regions) {
>>> 760:   shenandoah_assert_heaplocked();
>>> 761:   // This resets all state information, removing all regions from all partitions.
>> 
>> I thought it makes them all unavailable, placing them all into the NotFree partition.
>
> The wording has been a bit imprecise, possibly made even worse by some global search and replaces on free-set and partition.  I've tried to clarify in most recent draft that we consider Collector and Mutator to be partitions, but the NotFree labels means "not in a partition".  Maybe you can help me find the right wording here...

I'm adjusting this comment in hopes of making the intent more clear.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17561#discussion_r1477087368


More information about the hotspot-gc-dev mailing list