[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 04/20] target/arm: Move PC alignment check
From: |
Fabiano Rosas |
Subject: |
[PATCH 04/20] target/arm: Move PC alignment check |
Date: |
Wed, 18 Jan 2023 16:35:02 -0300 |
Move this earlier to make the next patch diff cleaner. While here
update the comment slightly to not give the impression that the
misalignment affects only TCG.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Fabiano Rosas <farosas@suse.de>
---
target/arm/machine.c | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/target/arm/machine.c b/target/arm/machine.c
index 5f26152652..b4c3850570 100644
--- a/target/arm/machine.c
+++ b/target/arm/machine.c
@@ -839,6 +839,15 @@ static int cpu_post_load(void *opaque, int version_id)
}
}
+ /*
+ * Misaligned thumb pc is architecturally impossible. Fail the
+ * incoming migration. For TCG it would trigger the assert in
+ * thumb_tr_translate_insn().
+ */
+ if (!is_a64(env) && env->thumb && (env->regs[15] & 1)) {
+ return -1;
+ }
+
hw_breakpoint_update_all(cpu);
hw_watchpoint_update_all(cpu);
@@ -856,15 +865,6 @@ static int cpu_post_load(void *opaque, int version_id)
}
}
- /*
- * Misaligned thumb pc is architecturally impossible.
- * We have an assert in thumb_tr_translate_insn to verify this.
- * Fail an incoming migrate to avoid this assert.
- */
- if (!is_a64(env) && env->thumb && (env->regs[15] & 1)) {
- return -1;
- }
-
if (!kvm_enabled()) {
pmu_op_finish(&cpu->env);
}
--
2.35.3
- [PATCH 00/20] target/arm: CONFIG_TCG=n part 1, Fabiano Rosas, 2023/01/18
- [PATCH 03/20] target/arm: wrap call to aarch64_sve_change_el in tcg_enabled(), Fabiano Rosas, 2023/01/18
- [PATCH 02/20] target/arm: wrap psci call with tcg_enabled, Fabiano Rosas, 2023/01/18
- [PATCH 01/20] target/arm: rename handle_semihosting to tcg_handle_semihosting, Fabiano Rosas, 2023/01/18
- [PATCH 09/20] target/arm: move translate modules to tcg/, Fabiano Rosas, 2023/01/18
- [PATCH 05/20] target/arm: Move cpregs code out of cpu.h, Fabiano Rosas, 2023/01/18
- [PATCH 08/20] target/arm: Wrap breakpoint/watchpoint updates with tcg_enabled, Fabiano Rosas, 2023/01/18
- [PATCH 06/20] target/arm: Move cpregs code into cpregs.c, Fabiano Rosas, 2023/01/18
- [PATCH 07/20] target/arm: Move define_debug_regs() to cpregs.c, Fabiano Rosas, 2023/01/18
- [PATCH 04/20] target/arm: Move PC alignment check,
Fabiano Rosas <=
- [PATCH 13/20] target/arm: Move hflags code into the tcg directory, Fabiano Rosas, 2023/01/18
- [PATCH 11/20] target/arm: Move psci.c into the tcg directory, Fabiano Rosas, 2023/01/18
- [PATCH 10/20] target/arm: move helpers to tcg/, Fabiano Rosas, 2023/01/18
- [PATCH 12/20] target/arm: Wrap arm_rebuild_hflags calls with tcg_enabled, Fabiano Rosas, 2023/01/18
- [PATCH 14/20] target/arm: Move regime_using_lpae_format into internal.h, Fabiano Rosas, 2023/01/18
- [PATCH 17/20] target/arm: Move cortex sysregs into cpregs.c, Fabiano Rosas, 2023/01/18
- [PATCH 15/20] target/arm: Don't access TCG code when debugging with KVM, Fabiano Rosas, 2023/01/18
- [PATCH 16/20] cpu-defs.h: Expose CPUTLBEntryFull to non-TCG code, Fabiano Rosas, 2023/01/18
- [PATCH 18/20] tests/qtest: Restrict bcm2835-dma-test to CONFIG_RASPI, Fabiano Rosas, 2023/01/18
- [PATCH 19/20] tests/avocado: Skip tests that require a missing accelerator, Fabiano Rosas, 2023/01/18