On 12/1/23 01:43, Richard Henderson wrote:
Unify the two helper_set_pstate_{sm,za} in this function. Do not call helper_* functions from svcr_write. Cleans up linux-user usage by consolodating logic.Cc: Fabiano Rosas <faro...@suse.de> Signed-off-by: Richard Henderson <richard.hender...@linaro.org> --- Fabiano, I expect this to replace much of your [RFC PATCH v2 07/19] target/arm: Move helper_set_pstate_* into cpregs.c r~ --- target/arm/cpu.h | 2 +- target/arm/helper-sme.h | 3 +-- linux-user/aarch64/cpu_loop.c | 11 ++-------- linux-user/aarch64/signal.c | 13 ++--------- target/arm/helper.c | 41 ++++++++++++++++++++++++++++++++--- target/arm/sme_helper.c | 37 ++----------------------------- target/arm/translate-a64.c | 19 ++++++---------- 7 files changed, 53 insertions(+), 73 deletions(-)
Since this patch was a bit too hard to digest at once, I split it in trivial steps here: https://lore.kernel.org/qemu-devel/20230112102436.1913-1-phi...@linaro.org/ For whichever version you prefer: Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
