All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Xuebing Wang <xbing6@gmail.com>, qemu-devel@nongnu.org
Cc: afaerber@suse.de, stefanha@redhat.com
Subject: Re: [Qemu-devel] [Discussion 08/10] exec: move TranslationBlock API from exec-all.h => translate.h
Date: Tue, 04 Mar 2014 11:27:09 +0100	[thread overview]
Message-ID: <5315AA7D.40204@redhat.com> (raw)
In-Reply-To: <1393901250-3922-9-git-send-email-xbing6@gmail.com>

Il 04/03/2014 03:47, Xuebing Wang ha scritto:
> Signed-off-by: Xuebing Wang <xbing6@gmail.com>
> ---
>  include/exec/exec-all.h  |  302 +--------------------------------------------
>  include/exec/translate.h |  306 ++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 307 insertions(+), 301 deletions(-)
>  create mode 100644 include/exec/translate.h

Very good idea.

Paolo

> diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h
> index a387922..68a888f 100644
> --- a/include/exec/exec-all.h
> +++ b/include/exec/exec-all.h
> @@ -20,307 +20,7 @@
>  #ifndef _EXEC_ALL_H_
>  #define _EXEC_ALL_H_
>
> -#include "qemu-common.h"
> -
> -/* allow to see translation results - the slowdown should be negligible, so we leave it */
> -#define DEBUG_DISAS
> -
> -/* Page tracking code uses ram addresses in system mode, and virtual
> -   addresses in userspace mode.  Define tb_page_addr_t to be an appropriate
> -   type.  */
> -#if defined(CONFIG_USER_ONLY)
> -typedef abi_ulong tb_page_addr_t;
> -#else
> -typedef ram_addr_t tb_page_addr_t;
> -#endif
> -
> -/* is_jmp field values */
> -#define DISAS_NEXT    0 /* next instruction can be analyzed */
> -#define DISAS_JUMP    1 /* only pc was modified dynamically */
> -#define DISAS_UPDATE  2 /* cpu state was modified dynamically */
> -#define DISAS_TB_JUMP 3 /* only pc was modified statically */
> -
> -struct TranslationBlock;
> -typedef struct TranslationBlock TranslationBlock;
> -
> -/* XXX: make safe guess about sizes */
> -#define MAX_OP_PER_INSTR 208
> -
> -#if HOST_LONG_BITS == 32
> -#define MAX_OPC_PARAM_PER_ARG 2
> -#else
> -#define MAX_OPC_PARAM_PER_ARG 1
> -#endif
> -#define MAX_OPC_PARAM_IARGS 5
> -#define MAX_OPC_PARAM_OARGS 1
> -#define MAX_OPC_PARAM_ARGS (MAX_OPC_PARAM_IARGS + MAX_OPC_PARAM_OARGS)
> -
> -/* A Call op needs up to 4 + 2N parameters on 32-bit archs,
> - * and up to 4 + N parameters on 64-bit archs
> - * (N = number of input arguments + output arguments).  */
> -#define MAX_OPC_PARAM (4 + (MAX_OPC_PARAM_PER_ARG * MAX_OPC_PARAM_ARGS))
> -#define OPC_BUF_SIZE 640
> -#define OPC_MAX_SIZE (OPC_BUF_SIZE - MAX_OP_PER_INSTR)
> -
> -/* Maximum size a TCG op can expand to.  This is complicated because a
> -   single op may require several host instructions and register reloads.
> -   For now take a wild guess at 192 bytes, which should allow at least
> -   a couple of fixup instructions per argument.  */
> -#define TCG_MAX_OP_SIZE 192
> -
> -#define OPPARAM_BUF_SIZE (OPC_BUF_SIZE * MAX_OPC_PARAM)
> -
> -#include "qemu/log.h"
> -
> -void gen_intermediate_code(CPUArchState *env, struct TranslationBlock *tb);
> -void gen_intermediate_code_pc(CPUArchState *env, struct TranslationBlock *tb);
> -void restore_state_to_opc(CPUArchState *env, struct TranslationBlock *tb,
> -                          int pc_pos);
> -
> -void cpu_gen_init(void);
> -int cpu_gen_code(CPUArchState *env, struct TranslationBlock *tb,
> -                 int *gen_code_size_ptr);
> -bool cpu_restore_state(CPUArchState *env, uintptr_t searched_pc);
> -void page_size_init(void);
> -
> -void QEMU_NORETURN cpu_resume_from_signal(CPUArchState *env1, void *puc);
> -void QEMU_NORETURN cpu_io_recompile(CPUArchState *env, uintptr_t retaddr);
> -TranslationBlock *tb_gen_code(CPUArchState *env,
> -                              target_ulong pc, target_ulong cs_base, int flags,
> -                              int cflags);
> -void cpu_exec_init(CPUArchState *env);
> -void QEMU_NORETURN cpu_loop_exit(CPUArchState *env1);
> -int page_unprotect(target_ulong address, uintptr_t pc, void *puc);
> -void tb_invalidate_phys_page_range(tb_page_addr_t start, tb_page_addr_t end,
> -                                   int is_cpu_write_access);
> -void tb_invalidate_phys_range(tb_page_addr_t start, tb_page_addr_t end,
> -                              int is_cpu_write_access);
> -#if !defined(CONFIG_USER_ONLY)
> -void tcg_cpu_address_space_init(CPUState *cpu, AddressSpace *as);
> -/* cputlb.c */
> -void tlb_flush_page(CPUArchState *env, target_ulong addr);
> -void tlb_flush(CPUArchState *env, int flush_global);
> -void tlb_set_page(CPUArchState *env, target_ulong vaddr,
> -                  hwaddr paddr, int prot,
> -                  int mmu_idx, target_ulong size);
> -void tb_invalidate_phys_addr(AddressSpace *as, hwaddr addr);
> -#else
> -static inline void tlb_flush_page(CPUArchState *env, target_ulong addr)
> -{
> -}
> -
> -static inline void tlb_flush(CPUArchState *env, int flush_global)
> -{
> -}
> -#endif
> -
> -#define CODE_GEN_ALIGN           16 /* must be >= of the size of a icache line */
> -
> -#define CODE_GEN_PHYS_HASH_BITS     15
> -#define CODE_GEN_PHYS_HASH_SIZE     (1 << CODE_GEN_PHYS_HASH_BITS)
> -
> -/* estimated block size for TB allocation */
> -/* XXX: use a per code average code fragment size and modulate it
> -   according to the host CPU */
> -#if defined(CONFIG_SOFTMMU)
> -#define CODE_GEN_AVG_BLOCK_SIZE 128
> -#else
> -#define CODE_GEN_AVG_BLOCK_SIZE 64
> -#endif
> -
> -#if defined(__arm__) || defined(_ARCH_PPC) \
> -    || defined(__x86_64__) || defined(__i386__) \
> -    || defined(__sparc__) || defined(__aarch64__) \
> -    || defined(CONFIG_TCG_INTERPRETER)
> -#define USE_DIRECT_JUMP
> -#endif
> -
> -struct TranslationBlock {
> -    target_ulong pc;   /* simulated PC corresponding to this block (EIP + CS base) */
> -    target_ulong cs_base; /* CS base for this block */
> -    uint64_t flags; /* flags defining in which context the code was generated */
> -    uint16_t size;      /* size of target code for this block (1 <=
> -                           size <= TARGET_PAGE_SIZE) */
> -    uint16_t cflags;    /* compile flags */
> -#define CF_COUNT_MASK  0x7fff
> -#define CF_LAST_IO     0x8000 /* Last insn may be an IO access.  */
> -
> -    uint8_t *tc_ptr;    /* pointer to the translated code */
> -    /* next matching tb for physical address. */
> -    struct TranslationBlock *phys_hash_next;
> -    /* first and second physical page containing code. The lower bit
> -       of the pointer tells the index in page_next[] */
> -    struct TranslationBlock *page_next[2];
> -    tb_page_addr_t page_addr[2];
> -
> -    /* the following data are used to directly call another TB from
> -       the code of this one. */
> -    uint16_t tb_next_offset[2]; /* offset of original jump target */
> -#ifdef USE_DIRECT_JUMP
> -    uint16_t tb_jmp_offset[2]; /* offset of jump instruction */
> -#else
> -    uintptr_t tb_next[2]; /* address of jump generated code */
> -#endif
> -    /* list of TBs jumping to this one. This is a circular list using
> -       the two least significant bits of the pointers to tell what is
> -       the next pointer: 0 = jmp_next[0], 1 = jmp_next[1], 2 =
> -       jmp_first */
> -    struct TranslationBlock *jmp_next[2];
> -    struct TranslationBlock *jmp_first;
> -    uint32_t icount;
> -};
> -
> -#include "exec/spinlock.h"
> -
> -typedef struct TBContext TBContext;
> -
> -struct TBContext {
> -
> -    TranslationBlock *tbs;
> -    TranslationBlock *tb_phys_hash[CODE_GEN_PHYS_HASH_SIZE];
> -    int nb_tbs;
> -    /* any access to the tbs or the page table must use this lock */
> -    spinlock_t tb_lock;
> -
> -    /* statistics */
> -    int tb_flush_count;
> -    int tb_phys_invalidate_count;
> -
> -    int tb_invalidated_flag;
> -};
> -
> -static inline unsigned int tb_jmp_cache_hash_page(target_ulong pc)
> -{
> -    target_ulong tmp;
> -    tmp = pc ^ (pc >> (TARGET_PAGE_BITS - TB_JMP_PAGE_BITS));
> -    return (tmp >> (TARGET_PAGE_BITS - TB_JMP_PAGE_BITS)) & TB_JMP_PAGE_MASK;
> -}
> -
> -static inline unsigned int tb_jmp_cache_hash_func(target_ulong pc)
> -{
> -    target_ulong tmp;
> -    tmp = pc ^ (pc >> (TARGET_PAGE_BITS - TB_JMP_PAGE_BITS));
> -    return (((tmp >> (TARGET_PAGE_BITS - TB_JMP_PAGE_BITS)) & TB_JMP_PAGE_MASK)
> -	    | (tmp & TB_JMP_ADDR_MASK));
> -}
> -
> -static inline unsigned int tb_phys_hash_func(tb_page_addr_t pc)
> -{
> -    return (pc >> 2) & (CODE_GEN_PHYS_HASH_SIZE - 1);
> -}
> -
> -void tb_free(TranslationBlock *tb);
> -void tb_flush(CPUArchState *env);
> -void tb_phys_invalidate(TranslationBlock *tb, tb_page_addr_t page_addr);
> -
> -#if defined(USE_DIRECT_JUMP)
> -
> -#if defined(CONFIG_TCG_INTERPRETER)
> -static inline void tb_set_jmp_target1(uintptr_t jmp_addr, uintptr_t addr)
> -{
> -    /* patch the branch destination */
> -    *(uint32_t *)jmp_addr = addr - (jmp_addr + 4);
> -    /* no need to flush icache explicitly */
> -}
> -#elif defined(_ARCH_PPC)
> -void ppc_tb_set_jmp_target(unsigned long jmp_addr, unsigned long addr);
> -#define tb_set_jmp_target1 ppc_tb_set_jmp_target
> -#elif defined(__i386__) || defined(__x86_64__)
> -static inline void tb_set_jmp_target1(uintptr_t jmp_addr, uintptr_t addr)
> -{
> -    /* patch the branch destination */
> -    *(uint32_t *)jmp_addr = addr - (jmp_addr + 4);
> -    /* no need to flush icache explicitly */
> -}
> -#elif defined(__aarch64__)
> -void aarch64_tb_set_jmp_target(uintptr_t jmp_addr, uintptr_t addr);
> -#define tb_set_jmp_target1 aarch64_tb_set_jmp_target
> -#elif defined(__arm__)
> -static inline void tb_set_jmp_target1(uintptr_t jmp_addr, uintptr_t addr)
> -{
> -#if !QEMU_GNUC_PREREQ(4, 1)
> -    register unsigned long _beg __asm ("a1");
> -    register unsigned long _end __asm ("a2");
> -    register unsigned long _flg __asm ("a3");
> -#endif
> -
> -    /* we could use a ldr pc, [pc, #-4] kind of branch and avoid the flush */
> -    *(uint32_t *)jmp_addr =
> -        (*(uint32_t *)jmp_addr & ~0xffffff)
> -        | (((addr - (jmp_addr + 8)) >> 2) & 0xffffff);
> -
> -#if QEMU_GNUC_PREREQ(4, 1)
> -    __builtin___clear_cache((char *) jmp_addr, (char *) jmp_addr + 4);
> -#else
> -    /* flush icache */
> -    _beg = jmp_addr;
> -    _end = jmp_addr + 4;
> -    _flg = 0;
> -    __asm __volatile__ ("swi 0x9f0002" : : "r" (_beg), "r" (_end), "r" (_flg));
> -#endif
> -}
> -#elif defined(__sparc__)
> -void tb_set_jmp_target1(uintptr_t jmp_addr, uintptr_t addr);
> -#else
> -#error tb_set_jmp_target1 is missing
> -#endif
> -
> -static inline void tb_set_jmp_target(TranslationBlock *tb,
> -                                     int n, uintptr_t addr)
> -{
> -    uint16_t offset = tb->tb_jmp_offset[n];
> -    tb_set_jmp_target1((uintptr_t)(tb->tc_ptr + offset), addr);
> -}
> -
> -#else
> -
> -/* set the jump target */
> -static inline void tb_set_jmp_target(TranslationBlock *tb,
> -                                     int n, uintptr_t addr)
> -{
> -    tb->tb_next[n] = addr;
> -}
> -
> -#endif
> -
> -static inline void tb_add_jump(TranslationBlock *tb, int n,
> -                               TranslationBlock *tb_next)
> -{
> -    /* NOTE: this test is only needed for thread safety */
> -    if (!tb->jmp_next[n]) {
> -        /* patch the native jump address */
> -        tb_set_jmp_target(tb, n, (uintptr_t)tb_next->tc_ptr);
> -
> -        /* add in TB jmp circular list */
> -        tb->jmp_next[n] = tb_next->jmp_first;
> -        tb_next->jmp_first = (TranslationBlock *)((uintptr_t)(tb) | (n));
> -    }
> -}
> -
> -/* GETRA is the true target of the return instruction that we'll execute,
> -   defined here for simplicity of defining the follow-up macros.  */
> -#if defined(CONFIG_TCG_INTERPRETER)
> -extern uintptr_t tci_tb_ptr;
> -# define GETRA() tci_tb_ptr
> -#else
> -# define GETRA() \
> -    ((uintptr_t)__builtin_extract_return_addr(__builtin_return_address(0)))
> -#endif
> -
> -/* The true return address will often point to a host insn that is part of
> -   the next translated guest insn.  Adjust the address backward to point to
> -   the middle of the call insn.  Subtracting one would do the job except for
> -   several compressed mode architectures (arm, mips) which set the low bit
> -   to indicate the compressed mode; subtracting two works around that.  It
> -   is also the case that there are no host isas that contain a call insn
> -   smaller than 4 bytes, so we don't worry about special-casing this.  */
> -#if defined(CONFIG_TCG_INTERPRETER)
> -# define GETPC_ADJ   0
> -#else
> -# define GETPC_ADJ   2
> -#endif
> -
> -#define GETPC()  (GETRA() - GETPC_ADJ)
> +#include "exec/translate.h"
>
>  #if !defined(CONFIG_USER_ONLY)
>
> diff --git a/include/exec/translate.h b/include/exec/translate.h
> new file mode 100644
> index 0000000..9e877f2
> --- /dev/null
> +++ b/include/exec/translate.h
> @@ -0,0 +1,306 @@
> +#ifndef EXEC_TRANSLATE_H
> +#define EXEC_TRANSLATE_H
> +
> +#include "qemu-common.h"
> +
> +/* allow to see translation results - the slowdown should be negligible, so we leave it */
> +#define DEBUG_DISAS
> +
> +/* Page tracking code uses ram addresses in system mode, and virtual
> +   addresses in userspace mode.  Define tb_page_addr_t to be an appropriate
> +   type.  */
> +#if defined(CONFIG_USER_ONLY)
> +typedef abi_ulong tb_page_addr_t;
> +#else
> +typedef ram_addr_t tb_page_addr_t;
> +#endif
> +
> +/* is_jmp field values */
> +#define DISAS_NEXT    0 /* next instruction can be analyzed */
> +#define DISAS_JUMP    1 /* only pc was modified dynamically */
> +#define DISAS_UPDATE  2 /* cpu state was modified dynamically */
> +#define DISAS_TB_JUMP 3 /* only pc was modified statically */
> +
> +struct TranslationBlock;
> +typedef struct TranslationBlock TranslationBlock;
> +
> +/* XXX: make safe guess about sizes */
> +#define MAX_OP_PER_INSTR 208
> +
> +#if HOST_LONG_BITS == 32
> +#define MAX_OPC_PARAM_PER_ARG 2
> +#else
> +#define MAX_OPC_PARAM_PER_ARG 1
> +#endif
> +#define MAX_OPC_PARAM_IARGS 5
> +#define MAX_OPC_PARAM_OARGS 1
> +#define MAX_OPC_PARAM_ARGS (MAX_OPC_PARAM_IARGS + MAX_OPC_PARAM_OARGS)
> +
> +/* A Call op needs up to 4 + 2N parameters on 32-bit archs,
> + * and up to 4 + N parameters on 64-bit archs
> + * (N = number of input arguments + output arguments).  */
> +#define MAX_OPC_PARAM (4 + (MAX_OPC_PARAM_PER_ARG * MAX_OPC_PARAM_ARGS))
> +#define OPC_BUF_SIZE 640
> +#define OPC_MAX_SIZE (OPC_BUF_SIZE - MAX_OP_PER_INSTR)
> +
> +/* Maximum size a TCG op can expand to.  This is complicated because a
> +   single op may require several host instructions and register reloads.
> +   For now take a wild guess at 192 bytes, which should allow at least
> +   a couple of fixup instructions per argument.  */
> +#define TCG_MAX_OP_SIZE 192
> +
> +#define OPPARAM_BUF_SIZE (OPC_BUF_SIZE * MAX_OPC_PARAM)
> +
> +#include "qemu/log.h"
> +
> +void gen_intermediate_code(CPUArchState *env, struct TranslationBlock *tb);
> +void gen_intermediate_code_pc(CPUArchState *env, struct TranslationBlock *tb);
> +void restore_state_to_opc(CPUArchState *env, struct TranslationBlock *tb,
> +                          int pc_pos);
> +
> +void cpu_gen_init(void);
> +int cpu_gen_code(CPUArchState *env, struct TranslationBlock *tb,
> +                 int *gen_code_size_ptr);
> +bool cpu_restore_state(CPUArchState *env, uintptr_t searched_pc);
> +void page_size_init(void);
> +
> +void QEMU_NORETURN cpu_resume_from_signal(CPUArchState *env1, void *puc);
> +void QEMU_NORETURN cpu_io_recompile(CPUArchState *env, uintptr_t retaddr);
> +TranslationBlock *tb_gen_code(CPUArchState *env,
> +                              target_ulong pc, target_ulong cs_base, int flags,
> +                              int cflags);
> +void cpu_exec_init(CPUArchState *env);
> +void QEMU_NORETURN cpu_loop_exit(CPUArchState *env1);
> +int page_unprotect(target_ulong address, uintptr_t pc, void *puc);
> +void tb_invalidate_phys_page_range(tb_page_addr_t start, tb_page_addr_t end,
> +                                   int is_cpu_write_access);
> +void tb_invalidate_phys_range(tb_page_addr_t start, tb_page_addr_t end,
> +                              int is_cpu_write_access);
> +#if !defined(CONFIG_USER_ONLY)
> +void tcg_cpu_address_space_init(CPUState *cpu, AddressSpace *as);
> +/* cputlb.c */
> +void tlb_flush_page(CPUArchState *env, target_ulong addr);
> +void tlb_flush(CPUArchState *env, int flush_global);
> +void tlb_set_page(CPUArchState *env, target_ulong vaddr,
> +                  hwaddr paddr, int prot,
> +                  int mmu_idx, target_ulong size);
> +void tb_invalidate_phys_addr(AddressSpace *as, hwaddr addr);
> +#else
> +static inline void tlb_flush_page(CPUArchState *env, target_ulong addr)
> +{
> +}
> +
> +static inline void tlb_flush(CPUArchState *env, int flush_global)
> +{
> +}
> +#endif
> +
> +#define CODE_GEN_ALIGN           16 /* must be >= of the size of a icache line */
> +
> +#define CODE_GEN_PHYS_HASH_BITS     15
> +#define CODE_GEN_PHYS_HASH_SIZE     (1 << CODE_GEN_PHYS_HASH_BITS)
> +
> +/* estimated block size for TB allocation */
> +/* XXX: use a per code average code fragment size and modulate it
> +   according to the host CPU */
> +#if defined(CONFIG_SOFTMMU)
> +#define CODE_GEN_AVG_BLOCK_SIZE 128
> +#else
> +#define CODE_GEN_AVG_BLOCK_SIZE 64
> +#endif
> +
> +#if defined(__arm__) || defined(_ARCH_PPC) \
> +    || defined(__x86_64__) || defined(__i386__) \
> +    || defined(__sparc__) || defined(__aarch64__) \
> +    || defined(CONFIG_TCG_INTERPRETER)
> +#define USE_DIRECT_JUMP
> +#endif
> +
> +struct TranslationBlock {
> +    target_ulong pc;   /* simulated PC corresponding to this block (EIP + CS base) */
> +    target_ulong cs_base; /* CS base for this block */
> +    uint64_t flags; /* flags defining in which context the code was generated */
> +    uint16_t size;      /* size of target code for this block (1 <=
> +                           size <= TARGET_PAGE_SIZE) */
> +    uint16_t cflags;    /* compile flags */
> +#define CF_COUNT_MASK  0x7fff
> +#define CF_LAST_IO     0x8000 /* Last insn may be an IO access.  */
> +
> +    uint8_t *tc_ptr;    /* pointer to the translated code */
> +    /* next matching tb for physical address. */
> +    struct TranslationBlock *phys_hash_next;
> +    /* first and second physical page containing code. The lower bit
> +       of the pointer tells the index in page_next[] */
> +    struct TranslationBlock *page_next[2];
> +    tb_page_addr_t page_addr[2];
> +
> +    /* the following data are used to directly call another TB from
> +       the code of this one. */
> +    uint16_t tb_next_offset[2]; /* offset of original jump target */
> +#ifdef USE_DIRECT_JUMP
> +    uint16_t tb_jmp_offset[2]; /* offset of jump instruction */
> +#else
> +    uintptr_t tb_next[2]; /* address of jump generated code */
> +#endif
> +    /* list of TBs jumping to this one. This is a circular list using
> +       the two least significant bits of the pointers to tell what is
> +       the next pointer: 0 = jmp_next[0], 1 = jmp_next[1], 2 =
> +       jmp_first */
> +    struct TranslationBlock *jmp_next[2];
> +    struct TranslationBlock *jmp_first;
> +    uint32_t icount;
> +};
> +
> +#include "exec/spinlock.h"
> +
> +typedef struct TBContext TBContext;
> +
> +struct TBContext {
> +
> +    TranslationBlock *tbs;
> +    TranslationBlock *tb_phys_hash[CODE_GEN_PHYS_HASH_SIZE];
> +    int nb_tbs;
> +    /* any access to the tbs or the page table must use this lock */
> +    spinlock_t tb_lock;
> +
> +    /* statistics */
> +    int tb_flush_count;
> +    int tb_phys_invalidate_count;
> +
> +    int tb_invalidated_flag;
> +};
> +
> +static inline unsigned int tb_jmp_cache_hash_page(target_ulong pc)
> +{
> +    target_ulong tmp;
> +    tmp = pc ^ (pc >> (TARGET_PAGE_BITS - TB_JMP_PAGE_BITS));
> +    return (tmp >> (TARGET_PAGE_BITS - TB_JMP_PAGE_BITS)) & TB_JMP_PAGE_MASK;
> +}
> +
> +static inline unsigned int tb_jmp_cache_hash_func(target_ulong pc)
> +{
> +    target_ulong tmp;
> +    tmp = pc ^ (pc >> (TARGET_PAGE_BITS - TB_JMP_PAGE_BITS));
> +    return (((tmp >> (TARGET_PAGE_BITS - TB_JMP_PAGE_BITS)) & TB_JMP_PAGE_MASK)
> +	    | (tmp & TB_JMP_ADDR_MASK));
> +}
> +
> +static inline unsigned int tb_phys_hash_func(tb_page_addr_t pc)
> +{
> +    return (pc >> 2) & (CODE_GEN_PHYS_HASH_SIZE - 1);
> +}
> +
> +void tb_free(TranslationBlock *tb);
> +void tb_flush(CPUArchState *env);
> +void tb_phys_invalidate(TranslationBlock *tb, tb_page_addr_t page_addr);
> +
> +#if defined(USE_DIRECT_JUMP)
> +
> +#if defined(CONFIG_TCG_INTERPRETER)
> +static inline void tb_set_jmp_target1(uintptr_t jmp_addr, uintptr_t addr)
> +{
> +    /* patch the branch destination */
> +    *(uint32_t *)jmp_addr = addr - (jmp_addr + 4);
> +    /* no need to flush icache explicitly */
> +}
> +#elif defined(_ARCH_PPC)
> +void ppc_tb_set_jmp_target(unsigned long jmp_addr, unsigned long addr);
> +#define tb_set_jmp_target1 ppc_tb_set_jmp_target
> +#elif defined(__i386__) || defined(__x86_64__)
> +static inline void tb_set_jmp_target1(uintptr_t jmp_addr, uintptr_t addr)
> +{
> +    /* patch the branch destination */
> +    *(uint32_t *)jmp_addr = addr - (jmp_addr + 4);
> +    /* no need to flush icache explicitly */
> +}
> +#elif defined(__aarch64__)
> +void aarch64_tb_set_jmp_target(uintptr_t jmp_addr, uintptr_t addr);
> +#define tb_set_jmp_target1 aarch64_tb_set_jmp_target
> +#elif defined(__arm__)
> +static inline void tb_set_jmp_target1(uintptr_t jmp_addr, uintptr_t addr)
> +{
> +#if !QEMU_GNUC_PREREQ(4, 1)
> +    register unsigned long _beg __asm ("a1");
> +    register unsigned long _end __asm ("a2");
> +    register unsigned long _flg __asm ("a3");
> +#endif
> +
> +    /* we could use a ldr pc, [pc, #-4] kind of branch and avoid the flush */
> +    *(uint32_t *)jmp_addr =
> +        (*(uint32_t *)jmp_addr & ~0xffffff)
> +        | (((addr - (jmp_addr + 8)) >> 2) & 0xffffff);
> +
> +#if QEMU_GNUC_PREREQ(4, 1)
> +    __builtin___clear_cache((char *) jmp_addr, (char *) jmp_addr + 4);
> +#else
> +    /* flush icache */
> +    _beg = jmp_addr;
> +    _end = jmp_addr + 4;
> +    _flg = 0;
> +    __asm __volatile__ ("swi 0x9f0002" : : "r" (_beg), "r" (_end), "r" (_flg));
> +#endif
> +}
> +#elif defined(__sparc__)
> +void tb_set_jmp_target1(uintptr_t jmp_addr, uintptr_t addr);
> +#else
> +#error tb_set_jmp_target1 is missing
> +#endif
> +
> +static inline void tb_set_jmp_target(TranslationBlock *tb,
> +                                     int n, uintptr_t addr)
> +{
> +    uint16_t offset = tb->tb_jmp_offset[n];
> +    tb_set_jmp_target1((uintptr_t)(tb->tc_ptr + offset), addr);
> +}
> +
> +#else
> +
> +/* set the jump target */
> +static inline void tb_set_jmp_target(TranslationBlock *tb,
> +                                     int n, uintptr_t addr)
> +{
> +    tb->tb_next[n] = addr;
> +}
> +
> +#endif
> +
> +static inline void tb_add_jump(TranslationBlock *tb, int n,
> +                               TranslationBlock *tb_next)
> +{
> +    /* NOTE: this test is only needed for thread safety */
> +    if (!tb->jmp_next[n]) {
> +        /* patch the native jump address */
> +        tb_set_jmp_target(tb, n, (uintptr_t)tb_next->tc_ptr);
> +
> +        /* add in TB jmp circular list */
> +        tb->jmp_next[n] = tb_next->jmp_first;
> +        tb_next->jmp_first = (TranslationBlock *)((uintptr_t)(tb) | (n));
> +    }
> +}
> +
> +/* GETRA is the true target of the return instruction that we'll execute,
> +   defined here for simplicity of defining the follow-up macros.  */
> +#if defined(CONFIG_TCG_INTERPRETER)
> +extern uintptr_t tci_tb_ptr;
> +# define GETRA() tci_tb_ptr
> +#else
> +# define GETRA() \
> +    ((uintptr_t)__builtin_extract_return_addr(__builtin_return_address(0)))
> +#endif
> +
> +/* The true return address will often point to a host insn that is part of
> +   the next translated guest insn.  Adjust the address backward to point to
> +   the middle of the call insn.  Subtracting one would do the job except for
> +   several compressed mode architectures (arm, mips) which set the low bit
> +   to indicate the compressed mode; subtracting two works around that.  It
> +   is also the case that there are no host isas that contain a call insn
> +   smaller than 4 bytes, so we don't worry about special-casing this.  */
> +#if defined(CONFIG_TCG_INTERPRETER)
> +# define GETPC_ADJ   0
> +#else
> +# define GETPC_ADJ   2
> +#endif
> +
> +#define GETPC()  (GETRA() - GETPC_ADJ)
> +
> +#endif /* EXEC_TRANSLATE_H */
>

  reply	other threads:[~2014-03-04 10:39 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-04  2:47 [Qemu-devel] [Discussion 00/10] about API hierarchy Xuebing Wang
2014-03-04  2:47 ` [Qemu-devel] [Discussion 01/10] docs: add docs/api-hierarchy.txt Xuebing Wang
2014-03-04  9:42   ` Stefan Hajnoczi
2014-03-04  9:58     ` Xuebing wang
2014-03-04 11:57       ` Stefan Hajnoczi
2014-03-04  2:47 ` [Qemu-devel] [Discussion 02/10] NEED_CPU_H: remove '#include "cpu.h"' from include/qemu-common.h Xuebing Wang
2014-03-04 10:19   ` Paolo Bonzini
2014-03-04 11:54     ` Xuebing wang
2014-03-04 12:02     ` Xuebing wang
2014-03-04 12:09       ` Paolo Bonzini
2014-03-04 12:09     ` Xuebing wang
2014-03-04 12:34       ` Peter Maydell
2014-03-04 12:40         ` Xuebing wang
2014-03-04 12:19     ` Xuebing wang
2014-03-04 12:23       ` Paolo Bonzini
2014-03-04 12:26     ` Xuebing wang
2014-03-04 12:29       ` Paolo Bonzini
2014-03-04  2:47 ` [Qemu-devel] [Discussion 03/10] NEED_CPU_H: remove unnecessary use of NEED_CPU_H Xuebing Wang
2014-03-04 10:20   ` Paolo Bonzini
2014-03-04  2:47 ` [Qemu-devel] [Discussion 04/10] memory_mapping: make this architecture-independent Xuebing Wang
2014-03-04 10:22   ` Paolo Bonzini
2014-03-04 11:05   ` Peter Maydell
2014-03-04  2:47 ` [Qemu-devel] [Discussion 05/10] NEED_CPU_H: remove unnecessary inclusion of "cpu.h" in root Xuebing Wang
2014-03-04 10:24   ` Paolo Bonzini
2014-03-04  2:47 ` [Qemu-devel] [Discussion 06/10] memory: move contents in include/exec/address-spaces.h => memory.h Xuebing Wang
2014-03-04 10:26   ` Paolo Bonzini
2014-03-04  2:47 ` [Qemu-devel] [Discussion 07/10] memory: remove file include/exec/address-spaces.h Xuebing Wang
2014-03-04  2:47 ` [Qemu-devel] [Discussion 08/10] exec: move TranslationBlock API from exec-all.h => translate.h Xuebing Wang
2014-03-04 10:27   ` Paolo Bonzini [this message]
2014-03-04  2:47 ` [Qemu-devel] [Discussion 09/10] exec: remove the unnecessary include of "exec-all.h" Xuebing Wang
2014-03-04 10:27   ` Paolo Bonzini
2014-03-04 11:11   ` Peter Maydell
2014-03-04 11:16     ` Peter Maydell
2014-03-04  2:47 ` [Qemu-devel] [Discussion 10/10] translate: remove file translate-all.h Xuebing Wang
2014-03-04 10:29   ` Paolo Bonzini
2014-03-04  3:45 ` [Qemu-devel] [Discussion 00/10] about API hierarchy Andreas Färber
2014-03-04  5:37   ` Xuebing wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5315AA7D.40204@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=afaerber@suse.de \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=xbing6@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.