RFR: Generation resizing [v4]

Y. Srinivas Ramakrishna ysr at openjdk.org
Fri Dec 9 01:11:19 UTC 2022


On Thu, 8 Dec 2022 23:57:57 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 17 commits:
>> 
>>  - Merge branch 'shenandoah-master' into mmu-instrumentation
>>  - Remove vestigial lock, do not enroll periodic task while holding threads_lock
>>  - Remove unnecessary logging, clean up imports
>>  - Merge from shenandoah/master
>>  - Document the class responsible for adjusting generation sizes
>>  - Revert unnecessary change
>>  - Remove unused time between cycle tracking
>>  - Remove vestigial mmu tracker instance
>>  - Clamp adjustments to min/max when increment is too large
>>  - Adjust generation sizes from safepoint
>>  - ... and 7 more: https://git.openjdk.org/shenandoah/compare/25469283...50896e31
>
> src/hotspot/share/gc/shenandoah/shenandoahMmuTracker.cpp line 99:
> 
>> 97:   _initial_process_time_s = process_time_s;
>> 98:   double verify_time_s = gc_thread_time_seconds();
>> 99:   double verify_elapsed = verify_time_s - _initial_verify_collector_time_s;
> 
> elapsed -> delta

Why do you use the `verify_` prefix here? I'm sure I am missing something here...

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

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


More information about the shenandoah-dev mailing list