RFR: 8318986: Improve GenericWaitBarrier performance [v3]
Aleksey Shipilev
shade at openjdk.org
Tue Nov 7 09:52:32 UTC 2023
On Tue, 7 Nov 2023 08:51:20 GMT, Robbin Ehn <rehn at openjdk.org> wrote:
>> Aleksey Shipilev has updated the pull request incrementally with four additional commits since the last revision:
>>
>> - Touchups
>> - More comments work
>> - Tight up the comments
>> - Rework to a single atomic counter per cell
>
> src/hotspot/share/utilities/waitBarrier_generic.cpp line 120:
>
>> 118: SpinYield sp;
>> 119: while (Atomic::load_acquire(&_state) < -1) {
>> 120: sp.wait();
>
> A warning once would be helpful that cells might be too few.
Yes, but I am a bit uncomfortable with printing performance warnings from a generic synchronization primitive. At very least because it might intermittently break some tests that check the outputs. So I'd rather avoid adding stuff here.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16404#discussion_r1384649730
More information about the hotspot-dev
mailing list