RFR: 8319797: Recursive lightweight locking: Runtime implementation [v4]

[David Holmes]

On Tue, 14 Nov 2023 08:16:38 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

> The current lock stack capacity is 8

Okay my recollection was they found 4 was sufficient, but I guess that changed.

> So if I understand you correctly, you want to inflate the current objects monitor unconditionally if the lock stack is full.

No I just expected checking for a full lock stack to happen when it is actually needed, not up front ie try to recursive lock, have it fail because the stack is full, then make some room and try again. The fullness check would be happening inside the lock stack code, not out in the caller. I'm not asking for you to do this just clarifying for you what I meant by not checking for it first.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16606#discussion_r1395366485