RFR: 8365880: Shenandoah: Unify memory usage accounting in ShenandoahFreeSet [v12]

Kelvin Nilsen kdnilsen at openjdk.org
Tue Oct 7 20:40:42 UTC 2025


On Thu, 2 Oct 2025 23:39:32 GMT, William Kemper <wkemper at openjdk.org> wrote:

>> Kelvin Nilsen has updated the pull request incrementally with five additional commits since the last revision:
>> 
>>  - small refactoring
>>  - OO refinements to used_regions_size()
>>  - fix broken assert from previous commit
>>  - OO refinements to free_unaffiliated_regions()
>>  - OO refinements to max_capacity()
>
> src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp line 745:
> 
>> 743: //  2. Region's free is less than minimum TLAB size and is retired
>> 744: //  3. The unused portion of memory in the last region of a humongous object
>> 745: void ShenandoahHeap::increase_used(const ShenandoahAllocRequest& req) {
> 
> I don't see the point of this method now. Can we delete it?

Removed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26867#discussion_r2411805067


More information about the hotspot-gc-dev mailing list