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

Kelvin Nilsen kdnilsen at openjdk.org
Wed Feb 7 21:25:12 UTC 2024


On Thu, 1 Feb 2024 02:58:51 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:

>> Kelvin Nilsen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Rename and comments for _capacity_of and _used_by
>
> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.hpp line 231:
> 
>> 229:   inline size_t available() const {
>> 230:     assert(used() <= capacity(), "must use less than capacity");
>> 231:     return capacity() - used();
> 
> So `ShenandoahFreeSet` publicly exports only the mutator view?

I think of this as "public to the mutator" and "friendly public to the collector".  I've tried to clarify with new comments.

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

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


More information about the hotspot-gc-dev mailing list