On Thu, 27 May 2021 15:19:59 GMT, Stefan Karlsson <[email protected]> wrote:
> I wonder if this should be changed to perform the read once by using > Atomic::load? That's the guidance we've given the last few years. Some > background for this: JDK-8234192. Hi Stefan, thanks for looking at it and thanks for the pointer. I do remember that Atomic::load should be preferred. But I would have to use it in thread.hpp and I don't know if I should change it in current_waiting_monitor() and current_pending_monitor(). Would it be acceptable to change the general code for this special usage? I'm not against doing it, I just want to double-check. It may improve reliability in general. ------------- PR: https://git.openjdk.java.net/jdk/pull/4224
