Hello Aurélien, This series picks up some preparatory patches for QOM CPUState refactoring, originally posted in May. They still applied cleanly, but I optimized them a bit and expanded the explanations.
In short it is about MIPSCPU vs. CPUMIPSState; more fields will be moved from CPU_COMMON macro to CPUState struct, and to access CPUState we need a MIPSCPU. Thus MIPSCPU is preferable for arguments of static helpers (not TCG helpers) because we save some redundant accessor/cast macros. Can you please ack/apply and keep in mind for the current patch review? Available for testing from: git://github.com/afaerber/qemu-cpu.git qom-cpu-mips https://github.com/afaerber/qemu-cpu/commits/qom-cpu-mips Regards, Andreas Cc: Aurélien Jarno <aurel...@aurel32.net> Cc: Jia Liu <pro...@gmail.com> Cc: Eric Johnson <er...@mips.com> Cc: Richard Henderson <r...@twiddle.net> v1 -> v2: * Cherry-picked from my CPUState part 4 series * Avoided calling mips_env_get_cpu() in both if branches of helper_m[t]tc0_tchalt() * Prepended patch to clean up resulting variable naming mess * Placed variable declarations in the closest block (requested by Alex elsewhere) Andreas Färber (5): target-mips: Clean up other_cpu in helper_{d,e}vpe() target-mips: Pass MIPSCPU to mips_tc_wake() target-mips: Pass MIPSCPU to mips_vpe_is_wfi() target-mips: Pass MIPSCPU to mips_tc_sleep() target-mips: Pass MIPSCPU to mips_vpe_sleep() target-mips/op_helper.c | 63 +++++++++++++++++++++++++++++------------------ 1 Datei geändert, 39 Zeilen hinzugefügt(+), 24 Zeilen entfernt(-) -- 1.7.10.4