aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Henderson <richard.henderson@linaro.org>2025-05-04 11:19:17 -0700
committerRichard Henderson <richard.henderson@linaro.org>2025-05-28 08:08:48 +0100
commit5c2891601ccdaa41427187ef95bc25c828b355e4 (patch)
tree24fa99493c25cc57fa98afaba1b1e86175da4930
parent90f80e4b0fedfc78163c1c112bb74ccbfcae2365 (diff)
downloadqemu-5c2891601ccdaa41427187ef95bc25c828b355e4.zip
qemu-5c2891601ccdaa41427187ef95bc25c828b355e4.tar.gz
qemu-5c2891601ccdaa41427187ef95bc25c828b355e4.tar.bz2
accel/tcg: Assert TCGCPUOps.pointer_wrap is set
All targets now provide the function, so we can make the call unconditional. Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
-rw-r--r--accel/tcg/cpu-exec.c1
-rw-r--r--accel/tcg/cputlb.c7
2 files changed, 3 insertions, 5 deletions
diff --git a/accel/tcg/cpu-exec.c b/accel/tcg/cpu-exec.c
index cc5f362..713bdb2 100644
--- a/accel/tcg/cpu-exec.c
+++ b/accel/tcg/cpu-exec.c
@@ -1039,6 +1039,7 @@ bool tcg_exec_realizefn(CPUState *cpu, Error **errp)
assert(tcg_ops->cpu_exec_halt);
assert(tcg_ops->cpu_exec_interrupt);
assert(tcg_ops->cpu_exec_reset);
+ assert(tcg_ops->pointer_wrap);
#endif /* !CONFIG_USER_ONLY */
assert(tcg_ops->translate_code);
assert(tcg_ops->get_tb_cpu_state);
diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c
index a734859..87e14bd 100644
--- a/accel/tcg/cputlb.c
+++ b/accel/tcg/cputlb.c
@@ -1773,11 +1773,8 @@ static bool mmu_lookup(CPUState *cpu, vaddr addr, MemOpIdx oi,
l->page[1].size = l->page[0].size - size0;
l->page[0].size = size0;
- if (cpu->cc->tcg_ops->pointer_wrap) {
- l->page[1].addr = cpu->cc->tcg_ops->pointer_wrap(cpu, l->mmu_idx,
- l->page[1].addr,
- addr);
- }
+ l->page[1].addr = cpu->cc->tcg_ops->pointer_wrap(cpu, l->mmu_idx,
+ l->page[1].addr, addr);
/*
* Lookup both pages, recognizing exceptions from either. If the