diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2019-11-06 12:33:10 +0100 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2019-11-06 13:36:32 +0100 |
| commit | 083244d320c3d8d7b93a08b2b0f5d06c6295be66 (patch) | |
| tree | fe40e95626e6b9641e959a8fce09387981b9fc0e | |
| parent | f8a745974de88779b000d4882b24a37153da2445 (diff) | |
| download | qemu-083244d320c3d8d7b93a08b2b0f5d06c6295be66.tar.gz | |
linux-user/sparc: Use WREG constants in sparc/target_cpu.h
This fixes a naming bug wherein we used "UREG_FP" to access the
stack pointer. OTOH, the "UREG_FP" constant was also defined
incorrectly such that it *did* reference the stack pointer.
Note that the kernel legitimately uses the name "FP", because it
utilizes the rolled stack window in processing the system call.
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <20191106113318.10226-5-richard.henderson@linaro.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
| -rw-r--r-- | linux-user/sparc/target_cpu.h | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/linux-user/sparc/target_cpu.h b/linux-user/sparc/target_cpu.h index 1ffc0ae9f2e..b30fbc72c43 100644 --- a/linux-user/sparc/target_cpu.h +++ b/linux-user/sparc/target_cpu.h @@ -41,15 +41,9 @@ static inline void cpu_set_tls(CPUSPARCState *env, target_ulong newtls) env->gregs[7] = newtls; } -#ifndef UREG_I6 -#define UREG_I6 6 -#endif -#ifndef UREG_FP -#define UREG_FP UREG_I6 -#endif - static inline abi_ulong get_sp_from_cpustate(CPUSPARCState *state) { - return state->regwptr[UREG_FP]; + return state->regwptr[WREG_SP]; } + #endif |