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

Kelvin Nilsen kdnilsen at openjdk.org
Sat Feb 3 14:29:03 UTC 2024


On Thu, 1 Feb 2024 07:15: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/shenandoahFullGC.cpp line 1069:
> 
>> 1067:     heap->collection_set()->clear();
>> 1068: 
>> 1069:     // Since Full GC directly manipulates top of certain regions, certain ShenandoahFreeSet abstractions may have been corrupted.
> 
> Instead of "may have been corrupted", which can be alarming and confusing, I'd state this as:
> 
> // Full GC doesn't use or maintain the ShenandoahFreeSet abstractions,
> // so we rebuild the free set from scratch following a Full GC.

I'm just going to remove that comment.  It raises concerns when none is necessary.  "Obviously", we need to rebuild the free set following a full gc.

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

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


More information about the hotspot-gc-dev mailing list