RFR: 8324649: Shenandoah: replace implementation of free set [v4]

[Y. Srinivas Ramakrishna]

On Wed, 31 Jan 2024 00:45:20 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp line 401:
>> 
>>> 399: 
>>> 400: HeapWord* ShenandoahFreeSet::allocate_single(ShenandoahAllocRequest& req, bool& in_new_region) {
>>> 401:   shenandoah_assert_heaplocked();
>> 
>> In addition, another precondition for this method appears to be that req.size() <= humongous size threshold. Perhaps that check should also be disposed of here. (Based on the documentation at the previous review comment above.)
>
> Added similar documentation here.  Thanks.

@kdnilsen ... bumping this up. I see that the documentation was added for precondition, but not the check/assert itself here in the implementation. Commenting to unresolve previously resolved comment.

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

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