[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 07/10] target/arm: Introduce ARM_FEATURE_V8_A
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH v3 07/10] target/arm: Introduce ARM_FEATURE_V8_ATOMICS and initial decode |
Date: |
Tue, 8 May 2018 10:31:48 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 05/08/2018 09:48 AM, Peter Maydell wrote:
> (A v8.x implementation can include any arbitrary
> subset of the v8.(x+1) features, so this is a legal combination.)
I didn't realize this was a possibility.
> In fact we already have a mandatory-in-v8.1 feature:
> ARM_FEATURE_V8_RDM, which we've just given its own feature bit.
Yes, I'd been planning to send follow-up patches to "tidy" this.
> --- a/target/arm/cpu.h
> +++ b/target/arm/cpu.h
> @@ -1449,8 +1449,7 @@ enum arm_features {
> ARM_FEATURE_V8_SHA3, /* implements SHA3 part of v8 Crypto Extensions */
> ARM_FEATURE_V8_SM3, /* implements SM3 part of v8 Crypto Extensions */
> ARM_FEATURE_V8_SM4, /* implements SM4 part of v8 Crypto Extensions */
> - ARM_FEATURE_V8_1,
> - ARM_FEATURE_V8_ATOMICS = ARM_FEATURE_V8_1, /* mandatory extension */
> + ARM_FEATURE_V8_ATOMICS, /* ARMv8.1-Atomics feature */
> ARM_FEATURE_V8_RDM, /* implements v8.1 simd round multiply */
> ARM_FEATURE_V8_FP16, /* implements v8.2 half-precision float */
> ARM_FEATURE_V8_FCMA, /* has complex number part of v8.3 extensions. */
>
> Is that OK?
Yes, that's fine. Thanks.
> We can have a separate patch which adds ARM_FEATURE_V8_1 and the
> code in realizefn to do
> if (arm_feature(env, ARM_FEATURE_V8_1)) {
> set_feature(env, ARM_FEATURE_V8);
> set_feature(env, ARM_FEATURE_V8_ATOMICS);
> set_feature(env, ARM_FEATURE_V8_RDM);
> }
Yep.
r~
- Re: [Qemu-devel] [PATCH v3 04/10] tcg: Introduce atomic helpers for integer min/max, (continued)
[Qemu-devel] [PATCH v3 08/10] target/arm: Fill in disas_ldst_atomic, Richard Henderson, 2018/05/08
[Qemu-devel] [PATCH v3 06/10] target/riscv: Use new atomic min/max expanders, Richard Henderson, 2018/05/08
[Qemu-devel] [PATCH v3 10/10] target/arm: Enable ARM_FEATURE_V8_ATOMICS for user-only, Richard Henderson, 2018/05/08
[Qemu-devel] [PATCH v3 09/10] target/arm: Implement CAS and CASP, Richard Henderson, 2018/05/08
[Qemu-devel] [PATCH v3 07/10] target/arm: Introduce ARM_FEATURE_V8_ATOMICS and initial decode, Richard Henderson, 2018/05/08
Re: [Qemu-devel] [PATCH v3 00/10] target/arm: Implement v8.1-Atomics, no-reply, 2018/05/08
Re: [Qemu-devel] [PATCH v3 00/10] target/arm: Implement v8.1-Atomics, Peter Maydell, 2018/05/10