Re: [PATCH 10/27] include/host: Split out atomic128-cas.h

[Philippe Mathieu-Daudé]

On 20/5/23 18:26, Richard Henderson wrote:
> Separates the aarch64-specific portion into its own file.
>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   host/include/aarch64/host/atomic128-cas.h | 43 ++++++++++++++++++
>   host/include/generic/host/atomic128-cas.h | 43 ++++++++++++++++++
>   include/qemu/atomic128.h                  | 55 +----------------------
>   3 files changed, 87 insertions(+), 54 deletions(-)
>   create mode 100644 host/include/aarch64/host/atomic128-cas.h
>   create mode 100644 host/include/generic/host/atomic128-cas.h
>
> diff --git a/host/include/aarch64/host/atomic128-cas.h 
> b/host/include/aarch64/host/atomic128-cas.h
> new file mode 100644
> index 0000000000..1247995419
> --- /dev/null
> +++ b/host/include/aarch64/host/atomic128-cas.h
> @@ -0,0 +1,43 @@
> +/*
> + * SPDX-License-Identifier: GPL-2.0-or-later
> + * Compare-and-swap for 128-bit atomic operations, generic version.

"Aarch64 specific"

> + *
> + * Copyright (C) 2018, 2023 Linaro, Ltd.
> + *
> + * See docs/devel/atomics.rst for discussion about the guarantees each
> + * atomic primitive is meant to provide.
> + */
> +
> +#ifndef AARCH64_ATOMIC128_CAS_H
> +#define AARCH64_ATOMIC128_CAS_H
> +
> +/* Through gcc 10, aarch64 has no support for 128-bit atomics.  */