RFR: 8315884: New Object to ObjectMonitor mapping [v6]
Daniel D. Daugherty
dcubed at openjdk.org
Tue Aug 13 22:16:10 UTC 2024
On Mon, 15 Jul 2024 00:45:10 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:
>> src/hotspot/share/runtime/lightweightSynchronizer.cpp line 401:
>>
>>> 399:
>>> 400: if (inserted) {
>>> 401: // Hopefully the performance counters are allocated on distinct
>>
>> It doesn't look like the counters are on distinct cache lines (see objectMonitor.hpp, lines 212ff). If this is a concern, file a bug to investigate it later? The comment here is a bit misplaced, IMO.
>
> It originates from https://github.com/openjdk/jdk/blob/15997bc3dfe9dddf21f20fa189f97291824892de/src/hotspot/share/runtime/synchronizer.cpp#L1543
>
> I think we just kept it and did not think more about it.
>
> Not sure what it is referring to. Maybe @dcubed-ojdk knows more, they originated from him (9 years old comment).
I don't think we ever got around to experimenting with putting the perf counters
on distinct cache lines. We've always had bigger fish to fry.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20067#discussion_r1715669185
More information about the core-libs-dev
mailing list