RFR: Handle old, pinned regions [v7]

William Kemper wkemper at openjdk.org
Fri Aug 5 23:07:55 UTC 2022


On Fri, 5 Aug 2022 07:25:52 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:

>> William Kemper has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 21 commits:
>> 
>>  - Merge branch 'shenandoah-master' into fill-old-pinned
>>  - Do not drop abbreviated argument when recording success concurrent cycle
>>  - Fix whitespace
>>  - Prepare for review
>>  - Merge branch 'shenandoah-master' into fill-old-pinned
>>  - Validate old gen state transitions
>>  - Make use of old generation without down casting
>>  - Make verifier aware of changes to old generation state machine
>>  - Make rset scan aware of changes to old generation state machine
>>  - Fix use of index as count and over-counting of garbage
>>  - ... and 11 more: https://git.openjdk.org/shenandoah/compare/60e43d04...bdda91d6
>
> src/hotspot/share/gc/shenandoah/heuristics/shenandoahOldHeuristics.cpp line 274:
> 
>> 272:                _last_old_collection_candidate,
>> 273:                byte_size_in_proper_unit(collectable_garbage), proper_unit_for_byte_size(collectable_garbage),
>> 274:                byte_size_in_proper_unit(immediate_garbage), proper_unit_for_byte_size(immediate_garbage));
> 
> See earlier comment about potentially underreporting `immediate_garbage`. Also, although `immediate_regions` is tracked, it doesn't seem to ever get used anywhere. May be it used to be reported at some point in the past but is no longer? Eliminate its tracking then?

I will report it in the log. We intend to use this value in the same fashion as young collections.

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

PR: https://git.openjdk.org/shenandoah/pull/149


More information about the shenandoah-dev mailing list