[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 06/13] target/arm: Derive id_isar4 from features
From: |
Richard Henderson |
Subject: |
[Qemu-devel] [PATCH 06/13] target/arm: Derive id_isar4 from features |
Date: |
Sat, 15 Sep 2018 09:17:31 -0700 |
??? The assertion does fire for the old cpus; they may be existing bugs.
??? Willfully provide a value for SWP_frac that matches our implementation.
Signed-off-by: Richard Henderson <address@hidden>
---
target/arm/cpu.c | 44 ++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/target/arm/cpu.c b/target/arm/cpu.c
index 2b199845fc..3c6ddd6532 100644
--- a/target/arm/cpu.c
+++ b/target/arm/cpu.c
@@ -887,6 +887,45 @@ static uint32_t resolve_id_isar3(CPUARMState *env)
return ret;
}
+static uint32_t resolve_id_isar4(CPUARMState *env)
+{
+ uint32_t ret = 0;
+
+ /* Unpriv -- note we don't support pre-armv4t. */
+ ret = deposit32(ret, 0, 4, arm_feature(env, ARM_FEATURE_THUMB2) ? 2 : 1);
+ /* WithShifts */
+ if (!arm_feature(env, ARM_FEATURE_M)) {
+ ret = deposit32(ret, 4, 4, 4);
+ } else if (arm_feature(env, ARM_FEATURE_V8)) {
+ ret = deposit32(ret, 4, 4, 3);
+ }
+ ret = deposit32(ret, 8, 4, 1); /* Writeback */
+ if (arm_feature(env, ARM_FEATURE_EL3)) {
+ /* Note that EL3 indicates Security Extensions. */
+ /* ??? In translate.c we check V6K instead. */
+ ret = deposit32(ret, 12, 4, 1); /* SMC */
+ }
+ if (arm_feature(env, ARM_FEATURE_V7)) {
+ ret = deposit32(ret, 16, 4, 1); /* Barrier */
+ }
+ if (!arm_feature(env, ARM_FEATURE_V6K) &&
+ arm_feature(env, ARM_FEATURE_V6)) {
+ ret = deposit32(ret, 20, 4, 3); /* SyncPrim_frac */
+ }
+ if (arm_feature(env, ARM_FEATURE_M)) {
+ ret = deposit32(ret, 24, 4, 1); /* PSR_M */
+ }
+ /*
+ * SWP_frac -- Value 1 indicates that SWP and SWPB only work in a
+ * uniprocessor context. Looking at ARM_FEATURE_SWP, we will have
+ * already set ID_ISAR0.Swap to 1, which means that SWP_frac must
+ * be ignored. While leaving this field 0 may not match certain
+ * real cpus, it is correct with respect to our implementation.
+ */
+
+ return ret;
+}
+
static void resolve_id_regs(ARMCPU *cpu)
{
CPUARMState *env = &cpu->env;
@@ -907,6 +946,11 @@ static void resolve_id_regs(ARMCPU *cpu)
orig = cpu->id_isar3;
cpu->id_isar3 = resolve_id_isar3(env);
g_assert_cmphex(cpu->id_isar3, ==, orig);
+
+ orig = cpu->id_isar4;
+ cpu->id_isar4 = resolve_id_isar4(env);
+ /* Willfully ignore the SWP_frac field. */
+ g_assert_cmphex(cpu->id_isar4 & 0x0fffffff, ==, orig & 0x0fffffff);
}
static void arm_cpu_realizefn(DeviceState *dev, Error **errp)
--
2.17.1
- [Qemu-devel] [RFC PATCH 00/13] target/arm: Derive cpu id regs from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 02/13] target/arm: Derive id_isar0 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 03/13] target/arm: Derive id_isar1 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 05/13] target/arm: Derive id_isar3 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 06/13] target/arm: Derive id_isar4 from features,
Richard Henderson <=
- [Qemu-devel] [PATCH 04/13] target/arm: Derive id_isar2 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 08/13] target/arm: Derive id_pfr0 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 07/13] target/arm: Derive id_isar5 and id_isar6 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 09/13] target/arm: Derive id_pfr1 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 10/13] target/arm: Derive id_aa64isar0 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 11/13] target/arm: Derive id_aa64isar1 from features, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 13/13] target/arm: Remove assertions from resolve_id_regs, Richard Henderson, 2018/09/15
- [Qemu-devel] [PATCH 12/13] target/arm: Derive id_aa64pfr0 from features, Richard Henderson, 2018/09/15
- Re: [Qemu-devel] [RFC PATCH 00/13] target/arm: Derive cpu id regs from features, Alex Bennée, 2018/09/19