Re: [PATCH 11/11] accel/tcg: Remove the now redundant cpu_exec_halt() hook

[Richard Henderson <richard.henderson@linaro.org>]

On 8/19/26 07:56, Philippe Mathieu-Daudé wrote:
> Every target setting the cpu_exec_halt callback alias it to
> their cpu_has_work() function, making the callback redundant.
> 
> Call cpu_has_work() directly in accel/tcg/cpu-exec.c
> cpu_has_work_after_processing_async_events() and remove the
> boilerplate cpu_exec_halt hook registration. No functional
> impact expected.
> 
> Signed-off-by: Philippe Mathieu-Daudé<philmd@oss.qualcomm.com>
> ---
>   include/accel/tcg/cpu-ops.h    | 17 -----------------
>   accel/tcg/cpu-exec.c           | 23 ++++++++---------------
>   target/alpha/cpu.c             |  1 -
>   target/avr/cpu.c               |  1 -
>   target/hexagon/cpu.c           |  1 -
>   target/hppa/cpu.c              |  1 -
>   target/loongarch/tcg/tcg_cpu.c |  1 -
>   target/m68k/cpu.c              |  1 -
>   target/microblaze/cpu.c        |  1 -
>   target/mips/cpu.c              |  1 -
>   target/or1k/cpu.c              |  1 -
>   target/ppc/cpu_init.c          |  1 -
>   target/riscv/tcg/tcg-cpu.c     |  1 -
>   target/rx/cpu.c                |  1 -
>   target/s390x/cpu.c             |  1 -
>   target/sh4/cpu.c               |  1 -
>   target/sparc/cpu.c             |  1 -
>   target/tricore/cpu.c           |  1 -
>   target/xtensa/cpu.c            |  1 -
>   19 files changed, 8 insertions(+), 49 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~