RFR: 8331411: Shenandoah: Reconsider spinning duration in ShenandoahLock [v7]

[Aleksey Shipilev]

On Mon, 8 Jul 2024 08:51:38 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> Xiaolong Peng has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Simplify code with less stacks
>
> src/hotspot/share/gc/shenandoah/shenandoahLock.cpp line 47:
> 
>> 45: void ShenandoahLock::contended_lock_internal(JavaThread* java_thread) {
>> 46:   assert(!ALLOW_BLOCK || java_thread != nullptr, "Must have a Java thread when allowing block.");
>> 47:   // Spin this much on multi-processor, do not spin on multi-processor.
> 
>>do not spin on multi-processor
> 
> Did you mean on **single**-processor?

Yes, I think so. @pengxiaolong, please do a simple follow-up fix?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19570#discussion_r1668426904