[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v3 2/4] target/arm: Add x-guarded-pages cpu property
From: |
Richard Henderson |
Subject: |
[Qemu-devel] [PATCH v3 2/4] target/arm: Add x-guarded-pages cpu property for user-only |
Date: |
Mon, 4 Feb 2019 13:12:26 +0000 |
While waiting for a proper userland ABI, allow static test
cases to be written assuming that GP is set for all pages.
Signed-off-by: Richard Henderson <address@hidden>
---
v2: Renamed the property with x- prefix
---
target/arm/cpu.h | 4 ++++
target/arm/cpu64.c | 20 ++++++++++++++++++++
target/arm/translate-a64.c | 8 +++++++-
3 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/target/arm/cpu.h b/target/arm/cpu.h
index 1ff7197efd..366ab97db3 100644
--- a/target/arm/cpu.h
+++ b/target/arm/cpu.h
@@ -900,6 +900,10 @@ struct ARMCPU {
*/
bool cfgend;
+#ifdef CONFIG_USER_ONLY
+ bool guarded_pages;
+#endif
+
QLIST_HEAD(, ARMELChangeHook) pre_el_change_hooks;
QLIST_HEAD(, ARMELChangeHook) el_change_hooks;
diff --git a/target/arm/cpu64.c b/target/arm/cpu64.c
index eff0f164dd..c5675fe7d1 100644
--- a/target/arm/cpu64.c
+++ b/target/arm/cpu64.c
@@ -281,6 +281,20 @@ static void cpu_max_set_sve_vq(Object *obj, Visitor *v,
const char *name,
error_propagate(errp, err);
}
+#ifdef CONFIG_USER_ONLY
+static bool aarch64_cpu_get_guarded_pages(Object *obj, Error **errp)
+{
+ ARMCPU *cpu = ARM_CPU(obj);
+ return cpu->guarded_pages;
+}
+
+static void aarch64_cpu_set_guarded_pages(Object *obj, bool val, Error **errp)
+{
+ ARMCPU *cpu = ARM_CPU(obj);
+ cpu->guarded_pages = val;
+}
+#endif
+
/* -cpu max: if KVM is enabled, like -cpu host (best possible with this host);
* otherwise, a CPU with as many features enabled as our emulation supports.
* The version of '-cpu max' for qemu-system-arm is defined in cpu.c;
@@ -360,6 +374,12 @@ static void aarch64_max_initfn(Object *obj)
*/
cpu->ctr = 0x80038003; /* 32 byte I and D cacheline size, VIPT icache
*/
cpu->dcz_blocksize = 7; /* 512 bytes */
+
+ object_property_add_bool(obj, "x-guarded-pages",
+ aarch64_cpu_get_guarded_pages,
+ aarch64_cpu_set_guarded_pages, NULL);
+ object_property_set_description(obj, "x-guarded-pages",
+ "Set on/off GuardPage bit for all pages", NULL);
#endif
cpu->sve_max_vq = ARM_MAX_VQ;
diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c
index 7375ebf7a9..37077138e3 100644
--- a/target/arm/translate-a64.c
+++ b/target/arm/translate-a64.c
@@ -13817,7 +13817,13 @@ static void disas_data_proc_simd_fp(DisasContext *s,
uint32_t insn)
static bool is_guarded_page(CPUARMState *env, DisasContext *s)
{
#ifdef CONFIG_USER_ONLY
- return false; /* FIXME */
+ /*
+ * FIXME: What is the userland ABI for this?
+ * For the moment this is controlled by an attribute:
+ * -cpu max,guarded_pages=on.
+ */
+ ARMCPU *cpu = arm_env_get_cpu(env);
+ return cpu->guarded_pages;
#else
uint64_t addr = s->base.pc_first;
int mmu_idx = arm_to_core_mmu_idx(s->mmu_idx);
--
2.17.2