On Fri, 25 Oct 2024 21:29:05 GMT, Patricio Chilano Mateo <pchilanom...@openjdk.org> wrote:
>> I see that in lightweightSynchronizer.cpp we already use the name >> `locking_thread` (although >> `LightweightSynchronizer::inflate_into_object_header` still uses >> `inflating_thread`). So how about using `locking_thread` instead? I can fix >> `LightweightSynchronizer::inflate_into_object_header` too. > >> If it's always the current thread, then it should be called 'current' imo. >> > The inflating thread is always the current one but it's not always equal to > `inflating_thread`. I thought locking_thread there may not be the current thread for enter_for() in deopt. It's the thread that should hold the lock but not the current thread. But it might be different now. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1817423564
