RFR: 8340490: Shenandoah: Optimize ShenandoahPacer [v2]

[Xiaolong Peng]

On Sat, 21 Sep 2024 05:52:10 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>>> I am good with this, assuming performance runs show good results.
>> 
>> Latency wise, in most time it is better than old impl. 
>> 
>> In my specific test with 8G heap on MacOS, throughput is very close to the test w/ ShenandoahPacing disabled, and about 25%~30% improvement comparing the old implementation.
>
>> > I am good with this, assuming performance runs show good results.
>> 
>> Latency wise, in most time it is better than old impl.
> 
> It is great it improves targeted tests, and it makes sense from the first principles. Run our usual performance pipeline to sanity check if this affects any other benchmarks in any meaningful way.

@shipilev Need you to review it again since I pushed minor refactor and format change as per your comments.

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

PR Comment: https://git.openjdk.org/jdk/pull/21099#issuecomment-2377706480