[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 05/28] translator: Use cpu_ld*_code instead of open-coding
From: |
Alex Bennée |
Subject: |
Re: [PATCH v2 05/28] translator: Use cpu_ld*_code instead of open-coding |
Date: |
Fri, 20 Dec 2019 17:12:21 +0000 |
User-agent: |
mu4e 1.3.5; emacs 27.0.50 |
Richard Henderson <address@hidden> writes:
> The DO_LOAD macros replicate the distinction already performed
> by the cpu_ldst.h functions. Use them.
>
> Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
> ---
> include/exec/cpu_ldst.h | 11 ---------
> include/exec/translator.h | 48 +++++++++++----------------------------
> 2 files changed, 13 insertions(+), 46 deletions(-)
>
> diff --git a/include/exec/cpu_ldst.h b/include/exec/cpu_ldst.h
> index cf8af36dbc..399ff6c3da 100644
> --- a/include/exec/cpu_ldst.h
> +++ b/include/exec/cpu_ldst.h
> @@ -129,11 +129,6 @@ static inline void clear_helper_retaddr(void)
> #include "exec/cpu_ldst_useronly_template.h"
> #undef MEMSUFFIX
>
> -/*
> - * Code access is deprecated in favour of translator_ld* functions
> - * (see translator.h). However there are still users that need to
> - * converted so for now these stay.
> - */
> #define MEMSUFFIX _code
> #define CODE_ACCESS
> #define DATA_SIZE 1
> @@ -455,12 +450,6 @@ void cpu_stq_mmuidx_ra(CPUArchState *env, abi_ptr addr,
> uint64_t val,
> #undef CPU_MMU_INDEX
> #undef MEMSUFFIX
>
> -/*
> - * Code access is deprecated in favour of translator_ld* functions
> - * (see translator.h). However there are still users that need to
> - * converted so for now these stay.
> - */
> -
> #define CPU_MMU_INDEX (cpu_mmu_index(env, true))
> #define MEMSUFFIX _code
> #define SOFTMMU_CODE_ACCESS
> diff --git a/include/exec/translator.h b/include/exec/translator.h
> index 459dd72aab..638e1529c5 100644
> --- a/include/exec/translator.h
> +++ b/include/exec/translator.h
> @@ -148,41 +148,19 @@ void translator_loop_temp_check(DisasContextBase *db);
> /*
> * Translator Load Functions
> *
> - * These are intended to replace the old cpu_ld*_code functions and
> - * are mandatory for front-ends that have been migrated to the common
> - * translator_loop. These functions are only intended to be called
> - * from the translation stage and should not be called from helper
> - * functions. Those functions should be converted to encode the
> - * relevant information at translation time.
> + * These are intended to replace the direct usage of the cpu_ld*_code
> + * functions and are mandatory for front-ends that have been migrated
> + * to the common translator_loop. These functions are only intended
> + * to be called from the translation stage and should not be called
> + * from helper functions. Those functions should be converted to encode
> + * the relevant information at translation time.
> */
>
> -#ifdef CONFIG_USER_ONLY
> -
> -#define DO_LOAD(type, name, shift) \
> - do { \
> - set_helper_retaddr(1); \
> - ret = name ## _p(g2h(pc)); \
> - clear_helper_retaddr(); \
> - } while (0)
> -
> -#else
> -
> -#define DO_LOAD(type, name, shift) \
> - do { \
> - int mmu_idx = cpu_mmu_index(env, true); \
> - TCGMemOpIdx oi = make_memop_idx(shift, mmu_idx); \
> - ret = helper_ret_ ## name ## _cmmu(env, pc, oi, 0); \
> - } while (0)
> -
> -#endif
> -
> -#define GEN_TRANSLATOR_LD(fullname, name, type, shift, swap_fn) \
> +#define GEN_TRANSLATOR_LD(fullname, type, load_fn, swap_fn) \
> static inline type \
> fullname ## _swap(CPUArchState *env, abi_ptr pc, bool do_swap) \
> { \
> - type ret; \
> - DO_LOAD(type, name, shift); \
> - \
> + type ret = load_fn(env, pc); \
> if (do_swap) { \
> ret = swap_fn(ret); \
> } \
> @@ -195,11 +173,11 @@ void translator_loop_temp_check(DisasContextBase *db);
> return fullname ## _swap(env, pc, false); \
> }
>
> -GEN_TRANSLATOR_LD(translator_ldub, ldub, uint8_t, 0, /* no swap */ )
> -GEN_TRANSLATOR_LD(translator_ldsw, ldsw, int16_t, 1, bswap16)
> -GEN_TRANSLATOR_LD(translator_lduw, lduw, uint16_t, 1, bswap16)
> -GEN_TRANSLATOR_LD(translator_ldl, ldl, uint32_t, 2, bswap32)
> -GEN_TRANSLATOR_LD(translator_ldq, ldq, uint64_t, 3, bswap64)
> +GEN_TRANSLATOR_LD(translator_ldub, uint8_t, cpu_ldub_code, /* no swap */)
> +GEN_TRANSLATOR_LD(translator_ldsw, int16_t, cpu_ldsw_code, bswap16)
> +GEN_TRANSLATOR_LD(translator_lduw, uint16_t, cpu_lduw_code, bswap16)
> +GEN_TRANSLATOR_LD(translator_ldl, uint32_t, cpu_ldl_code, bswap32)
> +GEN_TRANSLATOR_LD(translator_ldq, uint64_t, cpu_ldq_code, bswap64)
> #undef GEN_TRANSLATOR_LD
>
> #endif /* EXEC__TRANSLATOR_H */
--
Alex Bennée
- Re: [PATCH v2 02/28] cputlb: Use trace_mem_get_info instead of trace_mem_build_info, (continued)
- [PATCH v2 03/28] trace: Remove trace_mem_build_info_no_se_[bl]e, Richard Henderson, 2019/12/16
- [PATCH v2 04/28] cputlb: Move body of cpu_ldst_template.h out of line, Richard Henderson, 2019/12/16
- [PATCH v2 05/28] translator: Use cpu_ld*_code instead of open-coding, Richard Henderson, 2019/12/16
- Re: [PATCH v2 05/28] translator: Use cpu_ld*_code instead of open-coding,
Alex Bennée <=
- [PATCH v2 06/28] cputlb: Rename helper_ret_ld*_cmmu to cpu_ld*_code, Richard Henderson, 2019/12/16
- [PATCH v2 07/28] cputlb: Provide cpu_(ld, st}*_mmuidx_ra for user-only, Richard Henderson, 2019/12/16
- [PATCH v2 08/28] target/i386: Use cpu_*_mmuidx_ra instead of templates, Richard Henderson, 2019/12/16
- [PATCH v2 09/28] target/s390x: Include tcg.h in mem_helper.c, Richard Henderson, 2019/12/16