RFR: 8349748: [leyden] ciEnv::register_method should lock the correct queue lock
duke
duke at openjdk.org
Tue Feb 11 07:10:23 UTC 2025
On Mon, 10 Feb 2025 17:53:39 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Mainline code locks `MethodCompileQueue_lock` in `ciEnv::register_method`. But with Leyden-s `UseGlobalCompileQueueLock = false`, we have separate locks, per compile queue. So it seems to stand to reason we should be locking those locks, not a global one.
>
> I have not been able to attribute this gap to any bug or performance issue, but I think we better fix this proactively.
@shipilev
Your change (at version 04487caadaf47183b86eedc97b2de1a71ebbccfe) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/leyden/pull/37#issuecomment-2649982366
More information about the leyden-dev
mailing list