RFR: 8350314: Shenandoah: Capture thread state sync times in GC timings [v2]

William Kemper wkemper at openjdk.org
Tue Feb 25 23:49:54 UTC 2025


On Tue, 25 Feb 2025 23:45:21 GMT, William Kemper <wkemper at openjdk.org> wrote:

>> Xiaolong Peng has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address review comments
>
> src/hotspot/share/gc/shenandoah/shenandoahConcurrentGC.cpp line 1081:
> 
>> 1079:     heap->pacer()->setup_for_update_refs();
>> 1080:   }
>> 1081:   {
> 
> This one isn't necessary. This safepoint only happens when the pacer or the verifier is enabled. We moved init update ref gc state propagation into a handshake (see `ShenandoahHeap::concurrent_prepare_for_update_refs`). Sorry, I should have caught this in my first review.

I'm also currently working on removing the `final_roots` safepoint and fixing up the phase timings for the newish concurrent init update refs phase.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23759#discussion_r1970696214


More information about the hotspot-gc-dev mailing list