<font size=2 face="sans-serif">Hi All,</font><br><br><font size=2 face="sans-serif">Can I please request reviews for the
following change?</font><br><br><font size=2 face="sans-serif">Code change:</font><br><a href=http://cr.openjdk.java.net/~mdoerr/8155949_relaxed_cas/webrev.01/><font size=2 color=blue face="sans-serif">http://cr.openjdk.java.net/~mdoerr/8155949_relaxed_cas/webrev.01/</font></a><br><br><font size=2 face="sans-serif">This change follows the discussion started
from these mails.</font><br><a href="http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-April/018960.html"><font size=2 color=blue face="sans-serif">http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-April/018960.html</font></a><br><a href="http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-April/019148.html"><font size=2 color=blue face="sans-serif">http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-April/019148.html</font></a><br><a href="http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-May/019320.html"><font size=2 color=blue face="sans-serif">http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-May/019320.html</font></a><br><br><font size=2 face="sans-serif">Description:</font><br><font size=2 face="sans-serif">This change provides relaxed compare-and-exchange
by introducing</font><br><font size=2 face="sans-serif">relaxed memory order. As described in
atomic_linux_ppc.inline.hpp, </font><br><font size=2 face="sans-serif">the current implementation of cmpxchg
is fence_cmpxchg_acquire. </font><br><font size=2 face="sans-serif">This implementation is useful for general
purposes because twice calls of </font><br><font size=2 face="sans-serif">sync before and after cmpxchg will provide
strict consistency. </font><br><font size=2 face="sans-serif">However, they sometimes cause overheads
because sync instructions are </font><br><font size=2 face="sans-serif">very expensive in the current POWER
chip design.</font><br><br><font size=2 face="sans-serif">We confirmed this change improves performance
of copy_to_survivor</font><br><font size=2 face="sans-serif">in the parallel GC. However, we will
need more investigation of GC </font><br><font size=2 face="sans-serif">by more experts. So, We would like to
request a review of the change</font><br><font size=2 face="sans-serif">of cmpxchg first (as Martin requested).</font><br><a href="http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-April/019188.html"><font size=2 color=blue face="sans-serif">http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-April/019188.html</font></a><br><br><font size=2 face="sans-serif">Summary of source code changes:</font><br><br><font size=2 face="sans-serif">* src/share/vm/runtime/atomic.hpp </font><br><font size=2 face="sans-serif">     - Defines enum memory_order
and adds a parameter to cmpxchg.</font><br><br><font size=2 face="sans-serif">* src/share/vm/runtime/atomic.cpp</font><br><font size=2 face="sans-serif">* src/os_cpu/bsd_x86/vm/atomic_bsd_x86.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/bsd_zero/vm/atomic_bsd_zero.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/linux_aarch64/vm/atomic_linux_aarch64.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/linux_sparc/vm/atomic_linux_sparc.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/linux_x86/vm/atomic_linux_x86.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/linux_zero/vm/atomic_linux_zero.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/solaris_sparc/vm/atomic_solaris_sparc.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/solaris_x86/vm/atomic_solaris_x86.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/windows_x86/vm/atomic_windows_x86.inline.hpp</font><br><font size=2 face="sans-serif">     - Added a parameter
for each cmpxchg function to follow</font><br><font size=2 face="sans-serif">        the change
of atomic.hpp. Their implementations are not changed.</font><br><br><font size=2 face="sans-serif">* src/os_cpu/aix_ppc/vm/atomic_aix_ppc.inline.hpp</font><br><font size=2 face="sans-serif">* src/os_cpu/linux_ppc/vm/atomic_linux_ppc.inline.hpp</font><br><font size=2 face="sans-serif">     - Added a parameter
for each cmpxchg function to follow</font><br><font size=2 face="sans-serif">        the change
of atomic.hpp. In addition, implementations </font><br><font size=2 face="sans-serif">        are changed
corresponding to the specified memory_order.</font><br><br><font size=2 face="sans-serif">Regards,<br>Hiroshi<br>-----------------------<br>Hiroshi Horii, Ph.D.<br>IBM Research - Tokyo<br></font><BR>