RFR: 8291555: Implement alternative fast-locking scheme [v28]
Thomas Stuefe
stuefe at openjdk.org
Fri Mar 17 06:18:40 UTC 2023
On Thu, 16 Mar 2023 20:47:59 GMT, Roman Kennke <rkennke at openjdk.org> wrote:
> In my last changes I made a stupid mistake and don't set the condition flags correctly to force the slow-path, on aarch64. This is only relevant when we exceed the lock-stack capacity, that is why it's failing so rarely. I don't see a similar problem on x86_64 - have we observed any failures on x86_64? I pushed a fix for aarch64.
I noticed this too for arm; I used cmp to clear EQ but using tst seems better. I also do it inside fast_lock, to give it a defined exit state wrt EQ|NE, since it saves me from having to think about this on every call site. But at least the fail case may be fiddly without conditional execution.
-------------
PR: https://git.openjdk.org/jdk/pull/10907
More information about the serviceability-dev
mailing list