On Wed, 18 May 2022 19:05:03 GMT, Dmitry Chuyko <dchu...@openjdk.org> wrote:
> On AArch64 it is sometimes convenient to have LSE atomics right from the
> start. Currently they are enabled after feature detection and RR reverse
> debugger works incorrectly.
>
> New build configuration feature 'hardlse' is added. If it is enabled for
> aarch64 type of build, then statically compiled stubs replace the initial
> pessimistic implementation and dynamically generated replacements (when LSE
> support is detected). The feature works for builds of all debug levels.
>
> New file atomic_linux_aarch64_lse.S is derived from atomic_linux_aarch64.S
> and inherits its copyright. This alternative static implementation
> corresponds to the dynamically generated code.
>
> Note, this configuration part is necessary but not sufficient to fully avoid
> strex instructions for practical purposes. Other parts are:
>
> * Run on the OS built without strex family instructions. E.g. Amazon Linux
> 2022.
> * Compile with outline atomics enabled and the configuration flag enabled.
> E.g. configure with
> --with-extra-cflags='-march=armv8.3-a+crc+crypto -moutline-atomics'
> --with-extra-cxxflags='-march=armv8.3-a+crc+crypto -moutline-atomics'
> --with-extra-ldflags='-Wl,--allow-multiple-definition'
> --with-jvm-features=hardlse
>
> Testing: tier1, tier2 on linux-aarch64 release builds with feature off and
> feature on.
src/hotspot/os_cpu/linux_aarch64/atomic_linux_aarch64_lse.S line 30:
> 28: aarch64_atomic_fetch_add_8_default_impl:
> 29: prfm pstl1strm, [x0]
> 30: 0: ldaddal x1, x2, [x0]
Suggestion:
ldaddal x1, x2, [x0]
You don't need the `0:`label.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8779
