Re: [PATCH-for-9.1 10/12] exec/cpu-defs: Restrict SOFTMMU specific definitions to accel/tcg/

[Richard Henderson]

On 3/13/24 11:33, Philippe Mathieu-Daudé wrote:
> CPU_TLB_foo definitions are specific to SoftMMU and
> only used in accel/tcg/.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   accel/tcg/internal-target.h | 26 ++++++++++++++++++++++++++
>   include/exec/cpu-defs.h     | 26 --------------------------
>   2 files changed, 26 insertions(+), 26 deletions(-)
>
> diff --git a/accel/tcg/internal-target.h b/accel/tcg/internal-target.h
> index b22b29c461..9b5cc9168b 100644
> --- a/accel/tcg/internal-target.h
> +++ b/accel/tcg/internal-target.h
> @@ -12,6 +12,32 @@
>   #include "exec/exec-all.h"
>   #include "exec/translate-all.h"
>   
> +#if defined(CONFIG_SOFTMMU) && defined(CONFIG_TCG)

I see this is moved intact, but drop the CONFIG_TCG ifdef within accel/tcg/.
With that change,

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

r~