Re: [PATCH 33/33] include/exec: Change cpu_mmu_index argument to CPUState

[Philippe Mathieu-Daudé]

What is the rationale here? (I'm trying to generically understand
in which case one is better than another).

On 30/1/24 00:30, Richard Henderson wrote:
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   include/exec/cpu-all.h            |  2 +-
>   include/exec/cpu-common.h         |  3 +--
>   target/sparc/cpu.h                |  2 +-
>   accel/tcg/cputlb.c                | 22 +++++++++-------
>   semihosting/uaccess.c             |  2 +-
>   target/cris/translate.c           |  2 +-
>   target/hppa/mem_helper.c          |  2 +-
>   target/hppa/op_helper.c           |  8 +++---
>   target/i386/tcg/translate.c       |  2 +-
>   target/loongarch/tcg/tlb_helper.c |  4 +--
>   target/m68k/op_helper.c           |  2 +-
>   target/microblaze/helper.c        |  3 +--
>   target/microblaze/mmu.c           |  2 +-
>   target/microblaze/translate.c     |  2 +-
>   target/nios2/translate.c          |  2 +-
>   target/openrisc/translate.c       |  2 +-
>   target/sparc/ldst_helper.c        |  2 +-
>   target/sparc/mmu_helper.c         |  2 +-
>   target/tricore/helper.c           |  2 +-
>   target/tricore/translate.c        |  2 +-
>   target/xtensa/mmu_helper.c        |  2 +-
>   accel/tcg/ldst_common.c.inc       | 42 ++++++++++++++++++++-----------
>   22 files changed, 65 insertions(+), 49 deletions(-)