[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/1] target/arm: Fix GETPC usage in do_paired_cm
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH 3/1] target/arm: Fix GETPC usage in do_paired_cmpxchg64_l/be |
Date: |
Tue, 14 Nov 2017 16:11:08 +0000 |
User-agent: |
mu4e 1.0-alpha2; emacs 26.0.90 |
Richard Henderson <address@hidden> writes:
> Use of GETPC must be restricted to those functions that are
> directly called from TCG generated code.
>
> Fixes: 2399d4e7cec22ecf1c51062d2ebfd45220dbaace
> Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
> ---
> target/arm/helper-a64.c | 14 ++++++--------
> 1 file changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/target/arm/helper-a64.c b/target/arm/helper-a64.c
> index 96a3ecf707..b84ebcae6e 100644
> --- a/target/arm/helper-a64.c
> +++ b/target/arm/helper-a64.c
> @@ -432,9 +432,8 @@ uint64_t HELPER(crc32c_64)(uint64_t acc, uint64_t val,
> uint32_t bytes)
> /* Returns 0 on success; 1 otherwise. */
> static uint64_t do_paired_cmpxchg64_le(CPUARMState *env, uint64_t addr,
> uint64_t new_lo, uint64_t new_hi,
> - bool parallel)
> + bool parallel, uintptr_t ra)
> {
> - uintptr_t ra = GETPC();
> Int128 oldv, cmpv, newv;
> bool success;
>
> @@ -491,20 +490,19 @@ static uint64_t do_paired_cmpxchg64_le(CPUARMState
> *env, uint64_t addr,
> uint64_t HELPER(paired_cmpxchg64_le)(CPUARMState *env, uint64_t addr,
> uint64_t new_lo, uint64_t
> new_hi)
> {
> - return do_paired_cmpxchg64_le(env, addr, new_lo, new_hi, false);
> + return do_paired_cmpxchg64_le(env, addr, new_lo, new_hi, false, GETPC());
> }
>
> uint64_t HELPER(paired_cmpxchg64_le_parallel)(CPUARMState *env, uint64_t
> addr,
> uint64_t new_lo, uint64_t
> new_hi)
> {
> - return do_paired_cmpxchg64_le(env, addr, new_lo, new_hi, true);
> + return do_paired_cmpxchg64_le(env, addr, new_lo, new_hi, true, GETPC());
> }
>
> static uint64_t do_paired_cmpxchg64_be(CPUARMState *env, uint64_t addr,
> uint64_t new_lo, uint64_t new_hi,
> - bool parallel)
> + bool parallel, uintptr_t ra)
> {
> - uintptr_t ra = GETPC();
> Int128 oldv, cmpv, newv;
> bool success;
>
> @@ -561,11 +559,11 @@ static uint64_t do_paired_cmpxchg64_be(CPUARMState
> *env, uint64_t addr,
> uint64_t HELPER(paired_cmpxchg64_be)(CPUARMState *env, uint64_t addr,
> uint64_t new_lo, uint64_t new_hi)
> {
> - return do_paired_cmpxchg64_be(env, addr, new_lo, new_hi, false);
> + return do_paired_cmpxchg64_be(env, addr, new_lo, new_hi, false, GETPC());
> }
>
> uint64_t HELPER(paired_cmpxchg64_be_parallel)(CPUARMState *env, uint64_t
> addr,
> uint64_t new_lo, uint64_t new_hi)
> {
> - return do_paired_cmpxchg64_be(env, addr, new_lo, new_hi, true);
> + return do_paired_cmpxchg64_be(env, addr, new_lo, new_hi, true, GETPC());
> }
--
Alex Bennée