On Mon, 28 Oct 2024 00:38:39 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> 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. > > The thread passed in need not be the current thread, and IIUC is the thread > that should become the owner of the newly inflated monitor (either current > thread or a suspended thread). The actual inflation is always done by the > current thread. ok, I now I see what the discussion is. Yes I think locking_thread is better than inflating thread in this. Unless it's a bigger cleanup and we can do it post-integrating this. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1818935916
