On Fri, 28 May 2021 10:13:25 GMT, Martin Doerr <[email protected]> wrote:
>> We need a fix for crashes in get_current_contended_monitor due to concurrent >> modification of memory locations which are not declared volatile. See bug >> for details. > > Martin Doerr has updated the pull request incrementally with one additional > commit since the last revision: > > New solution: Use Atomic::load/store in current_pending/waiting_monitor > accessors. Thanks, folks, for reviewing! Changed to use Atomic::load/store. With this version, we use volatile accesses to these two members consistently. This may restrict compiler optimizations a bit, but I wouldn't expect a performance problem. ------------- PR: https://git.openjdk.java.net/jdk/pull/4224
