RFR: Add generations to freeset [v5]

Aleksey Shipilev shade at openjdk.org
Thu Apr 13 11:33:12 UTC 2023


On Thu, 13 Apr 2023 11:01:40 GMT, Aleksey Shipilev <shade 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.cpp line 65:
> 
>> 63:   assert (idx < _max, "index is sane: " SIZE_FORMAT " < " SIZE_FORMAT " (left: " SIZE_FORMAT ", right: " SIZE_FORMAT ")",
>> 64:           idx, _max, _mutator_leftmost, _mutator_rightmost);
>> 65:   assert(!_mutator_free_bitmap.at(idx) || (alloc_capacity(ShenandoahHeap::heap()->get_region(idx)) > 0),
> 
> Things like `(alloc_capacity(ShenandoahHeap::heap()->get_region(idx)) > 0)` are used often here, might just use a separate method for it.

Also, for the benefit of fastdebug builds performance (= faster testing), this can be something like:


bool is_free = _mutator_free_bitmap.at(idx);
assert(!is_free || has_alloc_capacity(idx), "Mutator free set should contain useful regions");
return is_free;

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/250#discussion_r1165366829


More information about the shenandoah-dev mailing list