RFR: 8267842: SIGSEGV in get_current_contended_monitor [v3]

[Martin Doerr]

On Fri, 28 May 2021 10:13:25 GMT, Martin Doerr <mdoerr at openjdk.org> 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