RFR: 8318986: Improve GenericWaitBarrier performance [v4]

Robbin Ehn rehn at openjdk.org
Wed Nov 8 07:12:59 UTC 2023


On Tue, 7 Nov 2023 20:18:11 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> src/hotspot/share/utilities/waitBarrier_generic.hpp line 38:
>> 
>>> 36:   private:
>>> 37:     DEFINE_PAD_MINUS_SIZE(0, DEFAULT_CACHE_LINE_SIZE, 0);
>>> 38: 
>> 
>> Just reading the padding, it's unclear why the two pads are where they are.
>> Can you add a comment about why choose these two locations?
>
> Wanted to make sure nothing interferes with cells. But now I realize we actually overpad between the cells (due to both pre-cell and post-cell padding), and underpad for the barrier itself! Fixed in new commit.

Ok!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16404#discussion_r1386104404


More information about the hotspot-dev mailing list