Re: [Qemu-devel] [PATCH v3 00/10] target/arm: Implement v8.1-Atomics

[Peter Maydell]

On 8 May 2018 at 16:14, Richard Henderson <address@hidden> wrote:
> This implements the Atomics extension, which is mandatory for v8.1.
> While testing the v8.2-SVE extension, I've run into issues with the
> GCC testsuite expecting this to exist.
>
> Missing is the wiring up of the system registers to indicate that
> the extension exists, but we have no system CPU model that would
> exercise such a setting.
>
> Changes since v3:
>   * Patch 8: Do not zero-extend X[s] via the third parameter
>     to read_cpu_reg.
>
> Changes since v2:
>   * New patch to use a helper macro for opposite-endian
>     atomic_fetch_add and atomic_add_fetch, as suggested
>     by pm215.
>   * Introduce ARM_FEATURE_V8_1 and define ARM_FEATURE_V8_ATOMICS
>     in terms of that, reinforcing the mandatory nature of
>     the extension.
>   * Typo fix in patch 8



Applied to target-arm.next (with the atomics.h workaround first
and the removal of ARM_FEATURE_V8_1), thanks.

-- PMM