On 8/3/21 6:13 AM, Richard Henderson wrote: > We shouldn't be ignoring SIGBUS for user-only. > > Move our existing TCGCPUOps hook out from CONFIG_SOFTMMU. > Move the wrapper, cpu_unaligned_access, to cpu-exec-common.c. > > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > --- > accel/tcg/internal.h | 4 ++++ > include/hw/core/tcg-cpu-ops.h | 16 ++++++++-------- > accel/tcg/cpu-exec-common.c | 12 ++++++++++++ > accel/tcg/cputlb.c | 9 --------- > 4 files changed, 24 insertions(+), 17 deletions(-) > > diff --git a/accel/tcg/internal.h b/accel/tcg/internal.h > index 881bc1ede0..a5e70cd91d 100644 > --- a/accel/tcg/internal.h > +++ b/accel/tcg/internal.h > @@ -19,4 +19,8 @@ void QEMU_NORETURN cpu_io_recompile(CPUState *cpu, > uintptr_t retaddr); > void page_init(void); > void tb_htable_init(void); > > +void QEMU_NORETURN cpu_unaligned_access(CPUState *cpu, vaddr addr, > + MMUAccessType access_type, > + int mmu_idx, uintptr_t retaddr);
Thanks for using noreturn :) Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>