Checks if the register is hidden.
Signed-off-by: Eric Auger <eric.auger@redhat.com>
---
target/arm/cpu.h | 2 ++
target/arm/cpu.c | 12 ++++++++++++
target/arm/kvm.c | 23 ++---------------------
3 files changed, 16 insertions(+), 21 deletions(-)
diff --git a/target/arm/cpu.h b/target/arm/cpu.h
index 30d59a51d6..3ae4d65422 100644
--- a/target/arm/cpu.h
+++ b/target/arm/cpu.h
@@ -2687,4 +2687,6 @@ static inline bool is_fake_reg(ARMCPU *cpu, uint64_t regidx)
#define LOG2_TAG_GRANULE 4
#define TAG_GRANULE (1 << LOG2_TAG_GRANULE)
+bool arm_cpu_hide_reg(CPUState *s, uint64_t regidx);
+
#endif
diff --git a/target/arm/cpu.c b/target/arm/cpu.c
index 3b556f1404..60eee82742 100644
--- a/target/arm/cpu.c
+++ b/target/arm/cpu.c
@@ -2366,6 +2366,17 @@ static const TCGCPUOps arm_tcg_ops = {
};
#endif /* CONFIG_TCG */
+bool arm_cpu_hide_reg(CPUState *s, uint64_t regidx)
+{
+ ARMCPU *cpu = ARM_CPU(s);
+ for (int i = 0; i < cpu->nr_kvm_hidden_regs; i++) {
+ if (cpu->kvm_hidden_regs[i] == regidx) {
+ return true;
+ }
+ }
+ return false;
+}
+
static void arm_cpu_class_init(ObjectClass *oc, const void *data)
{
ARMCPUClass *acc = ARM_CPU_CLASS(oc);
@@ -2394,6 +2405,7 @@ static void arm_cpu_class_init(ObjectClass *oc, const void *data)
cc->gdb_get_core_xml_file = arm_gdb_get_core_xml_file;
cc->gdb_stop_before_watchpoint = true;
cc->disas_set_info = arm_disas_set_info;
+ cc->hide_reg = arm_cpu_hide_reg;
#ifdef CONFIG_TCG
cc->tcg_ops = &arm_tcg_ops;
diff --git a/target/arm/kvm.c b/target/arm/kvm.c
index 7551c43e79..1a95e2c667 100644
--- a/target/arm/kvm.c
+++ b/target/arm/kvm.c
@@ -754,25 +754,6 @@ static bool kvm_arm_reg_syncs_via_cpreg_list(uint64_t regidx)
}
}
-/**
- * kvm_vcpu_compat_hidden_reg:
- * @cpu: ARMCPU
- * @regidx: index of the register to check
- *
- * Depending on the CPU compat returns true if @regidx must be
- * ignored during sync & migration
- */
-static inline bool
-kvm_vcpu_compat_hidden_reg(ARMCPU *cpu, uint64_t regidx)
-{
- for (int i = 0; i < cpu->nr_kvm_hidden_regs; i++) {
- if (cpu->kvm_hidden_regs[i] == regidx) {
- return true;
- }
- }
- return false;
-}
-
/**
* kvm_arm_init_cpreg_list:
* @cpu: ARMCPU
@@ -834,7 +815,7 @@ static int kvm_arm_init_cpreg_list(ARMCPU *cpu)
uint64_t regidx = rlp->reg[i];
if (!kvm_arm_reg_syncs_via_cpreg_list(regidx) ||
- kvm_vcpu_compat_hidden_reg(cpu, regidx)) {
+ arm_cpu_hide_reg(&cpu->parent_obj, regidx)) {
continue;
}
switch (rlp->reg[i] & KVM_REG_SIZE_MASK) {
@@ -867,7 +848,7 @@ static int kvm_arm_init_cpreg_list(ARMCPU *cpu)
if (!kvm_arm_reg_syncs_via_cpreg_list(regidx)) {
continue;
}
- if (kvm_vcpu_compat_hidden_reg(cpu, regidx)) {
+ if (arm_cpu_hide_reg(&cpu->parent_obj, regidx)) {
trace_kvm_arm_init_cpreg_list_skip_hidden_reg(rlp->reg[i]);
continue;
}
--
2.49.0