On Sat, Nov 6, 2021, 5:39 AM Philippe Mathieu-Daudé <f4...@amsat.org> wrote:
> cpu_loop() never exits, so mark it with QEMU_NORETURN. > > Reviewed-by: Richard Henderson <richard.hender...@linaro.org> > Reviewed-By: Warner Losh <i...@bsdimp.com> > Reviewed-by: Bin Meng <bmeng...@gmail.com> > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > --- > v2: > - rebased > - restricted to linux-user > > Supersedes: <20210905000429.1097336-1-f4...@amsat.org> > Reviewed-by: Warner Losh <i...@bsdimp.com> bsd-user likely needs similar treatment, no? --- > linux-user/user-internals.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/linux-user/user-internals.h b/linux-user/user-internals.h > index 661612a088b..c7ad00268af 100644 > --- a/linux-user/user-internals.h > +++ b/linux-user/user-internals.h > @@ -65,7 +65,7 @@ abi_long do_syscall(void *cpu_env, int num, abi_long > arg1, > abi_long arg5, abi_long arg6, abi_long arg7, > abi_long arg8); > extern __thread CPUState *thread_cpu; > -void cpu_loop(CPUArchState *env); > +void QEMU_NORETURN cpu_loop(CPUArchState *env); > const char *target_strerror(int err); > int get_osversion(void); > void init_qemu_uname_release(void); > -- > 2.31.1 > >
