RFR: Usage tracking cleanup [v2]

William Kemper wkemper at openjdk.org
Thu Apr 20 15:21:41 UTC 2023


On Wed, 19 Apr 2023 23:56:10 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 12 commits:
>> 
>>  - Track total heap usage in global generation
>>  - Remove unused method: clear_used
>>  - Readability improvements
>>  - Merge branch 'shenandoah-master' into alloc-tracking-cleanup
>>  - Merge branch 'shenandoah-master' into alloc-tracking-cleanup
>>  - Remove invalid assert
>>  - Merge branch 'shenandoah-master' into alloc-tracking-cleanup
>>  - Consolidate usage/allocate/waste accounting
>>  - WIP: Oops - report usage in bytes, not words
>>  - WIP: Initialize new padding field
>>  - ... and 2 more: https://git.openjdk.org/shenandoah/compare/016bf071...99c52914
>
> src/hotspot/share/gc/shenandoah/shenandoahFullGC.cpp line 1363:
> 
>> 1361: 
>> 1362:     // In generational mode, global usage should be the sum of young and old. This is also true
>> 1363:     // for non-generational modes except that there are no old regions.
> 
> ... so the first addend (or, augend) in each of the arguments below is 0 in the non-generational case? If so, is that worth asserting, just to be super-paranoid?

Yes - the assertions on 1357-1359 assert that these values are all zero for old generation in the non-generational modes.

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/260#discussion_r1172750426


More information about the shenandoah-dev mailing list