RFR: Add generations to freeset [v5]
Kelvin Nilsen
kdnilsen at openjdk.org
Tue Apr 18 22:46:25 UTC 2023
On Thu, 13 Apr 2023 02:16:43 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:
>>
>> Remove debug instrumentation
>
> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.hpp line 83:
>
>> 81: HeapWord* allocate_old_with_affiliation(ShenandoahAffiliation affiliation, ShenandoahAllocRequest& req, bool& in_new_region);
>> 82:
>> 83: // While holding the heap lock, allocate memory for a single object which is to be entirely contained
>
> I see that the callers of `allocate_...()` assert that the heap is locked, but would it be a good idea to also have these `allocate_...()` methods do so as well? May be I am being paranoid.
I've added the asserts. It's a good improvement.
-------------
PR Review Comment: https://git.openjdk.org/shenandoah/pull/250#discussion_r1170634154
More information about the shenandoah-dev
mailing list