target/arm/sme: Rebuild hflags in aarch64_set_svcr()

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Fabiano Rosas <farosas@suse.de>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-id: 20230112102436.1913-7-philmd@linaro.org
Message-Id: <20230112004322.161330-1-richard.henderson@linaro.org>
[PMD: Split patch in multiple tiny steps]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
Richard Henderson 2023-01-12 11:24:35 +01:00 committed by Peter Maydell
parent fccb49182e
commit f431855714
4 changed files with 6 additions and 19 deletions

View File

@ -89,14 +89,8 @@ void cpu_loop(CPUARMState *env)
switch (trapnr) { switch (trapnr) {
case EXCP_SWI: case EXCP_SWI:
/* /* On syscall, PSTATE.ZA is preserved, PSTATE.SM is cleared. */
* On syscall, PSTATE.ZA is preserved, along with the ZA matrix.
* PSTATE.SM is cleared, per SMSTOP, which does ResetSVEState.
*/
aarch64_set_svcr(env, 0, R_SVCR_SM_MASK); aarch64_set_svcr(env, 0, R_SVCR_SM_MASK);
if (FIELD_EX64(env->svcr, SVCR, SM)) {
arm_rebuild_hflags(env);
}
ret = do_syscall(env, ret = do_syscall(env,
env->xregs[8], env->xregs[8],
env->xregs[0], env->xregs[0],

View File

@ -667,9 +667,6 @@ static void target_setup_frame(int usig, struct target_sigaction *ka,
/* Invoke the signal handler with both SM and ZA disabled. */ /* Invoke the signal handler with both SM and ZA disabled. */
aarch64_set_svcr(env, 0, R_SVCR_SM_MASK | R_SVCR_ZA_MASK); aarch64_set_svcr(env, 0, R_SVCR_SM_MASK | R_SVCR_ZA_MASK);
if (env->svcr) {
arm_rebuild_hflags(env);
}
if (info) { if (info) {
tswap_siginfo(&frame->info, info); tswap_siginfo(&frame->info, info);

View File

@ -6738,6 +6738,9 @@ void aarch64_set_svcr(CPUARMState *env, uint64_t new, uint64_t mask)
{ {
uint64_t change = (env->svcr ^ new) & mask; uint64_t change = (env->svcr ^ new) & mask;
if (change == 0) {
return;
}
env->svcr ^= change; env->svcr ^= change;
if (change & R_SVCR_SM_MASK) { if (change & R_SVCR_SM_MASK) {
@ -6755,6 +6758,8 @@ void aarch64_set_svcr(CPUARMState *env, uint64_t new, uint64_t mask)
if (change & new & R_SVCR_ZA_MASK) { if (change & new & R_SVCR_ZA_MASK) {
memset(env->zarray, 0, sizeof(env->zarray)); memset(env->zarray, 0, sizeof(env->zarray));
} }
arm_rebuild_hflags(env);
} }
static void svcr_write(CPUARMState *env, const ARMCPRegInfo *ri, static void svcr_write(CPUARMState *env, const ARMCPRegInfo *ri,
@ -6763,7 +6768,6 @@ static void svcr_write(CPUARMState *env, const ARMCPRegInfo *ri,
helper_set_pstate_sm(env, FIELD_EX64(value, SVCR, SM)); helper_set_pstate_sm(env, FIELD_EX64(value, SVCR, SM));
helper_set_pstate_za(env, FIELD_EX64(value, SVCR, ZA)); helper_set_pstate_za(env, FIELD_EX64(value, SVCR, ZA));
aarch64_set_svcr(env, value, -1); aarch64_set_svcr(env, value, -1);
arm_rebuild_hflags(env);
} }
static void smcr_write(CPUARMState *env, const ARMCPRegInfo *ri, static void smcr_write(CPUARMState *env, const ARMCPRegInfo *ri,

View File

@ -31,20 +31,12 @@
void helper_set_pstate_sm(CPUARMState *env, uint32_t i) void helper_set_pstate_sm(CPUARMState *env, uint32_t i)
{ {
if (i == FIELD_EX64(env->svcr, SVCR, SM)) {
return;
}
aarch64_set_svcr(env, 0, R_SVCR_SM_MASK); aarch64_set_svcr(env, 0, R_SVCR_SM_MASK);
arm_rebuild_hflags(env);
} }
void helper_set_pstate_za(CPUARMState *env, uint32_t i) void helper_set_pstate_za(CPUARMState *env, uint32_t i)
{ {
if (i == FIELD_EX64(env->svcr, SVCR, ZA)) {
return;
}
aarch64_set_svcr(env, 0, R_SVCR_ZA_MASK); aarch64_set_svcr(env, 0, R_SVCR_ZA_MASK);
arm_rebuild_hflags(env);
} }
void helper_sme_zero(CPUARMState *env, uint32_t imm, uint32_t svl) void helper_sme_zero(CPUARMState *env, uint32_t imm, uint32_t svl)