[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] target-xtensa: switch to AREG0-free mode
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] target-xtensa: switch to AREG0-free mode |
Date: |
Sat, 9 Jun 2012 11:33:02 +0000 |
On Sun, Jun 3, 2012 at 9:06 PM, Max Filippov <address@hidden> wrote:
> Add env parameter to every helper function that needs it, update
> 'configure' script.
>
> Signed-off-by: Max Filippov <address@hidden>
> ---
> configure | 2 +-
> target-xtensa/helper.h | 54 +++++++-------
> target-xtensa/op_helper.c | 185
> ++++++++++++++++++++++-----------------------
> target-xtensa/translate.c | 64 ++++++++-------
The patch does not apply, also the following line in
target-xtensa/Makefile.objs needs to be deleted:
$(obj)/op_helper.o: QEMU_CFLAGS += $(HELPER_CFLAGS)
> 4 files changed, 152 insertions(+), 153 deletions(-)
>
> diff --git a/configure b/configure
> index 1f338f8..25c3acc 100755
> --- a/configure
> +++ b/configure
> @@ -3678,7 +3678,7 @@ case "$target_arch2" in
> esac
>
> case "$target_arch2" in
> - alpha | sparc*)
> + alpha | sparc* | xtensa*)
> echo "CONFIG_TCG_PASS_AREG0=y" >> $config_target_mak
> ;;
> esac
> diff --git a/target-xtensa/helper.h b/target-xtensa/helper.h
> index 9dd6f89..152fec0 100644
> --- a/target-xtensa/helper.h
> +++ b/target-xtensa/helper.h
> @@ -1,39 +1,39 @@
> #include "def-helper.h"
>
> -DEF_HELPER_1(exception, noreturn, i32)
> -DEF_HELPER_2(exception_cause, noreturn, i32, i32)
> -DEF_HELPER_3(exception_cause_vaddr, noreturn, i32, i32, i32)
> -DEF_HELPER_2(debug_exception, noreturn, i32, i32)
> +DEF_HELPER_2(exception, noreturn, env, i32)
> +DEF_HELPER_3(exception_cause, noreturn, env, i32, i32)
> +DEF_HELPER_4(exception_cause_vaddr, noreturn, env, i32, i32, i32)
> +DEF_HELPER_3(debug_exception, noreturn, env, i32, i32)
>
> DEF_HELPER_FLAGS_1(nsa, TCG_CALL_CONST | TCG_CALL_PURE, i32, i32)
> DEF_HELPER_FLAGS_1(nsau, TCG_CALL_CONST | TCG_CALL_PURE, i32, i32)
> -DEF_HELPER_1(wsr_windowbase, void, i32)
> -DEF_HELPER_3(entry, void, i32, i32, i32)
> -DEF_HELPER_1(retw, i32, i32)
> -DEF_HELPER_1(rotw, void, i32)
> -DEF_HELPER_2(window_check, void, i32, i32)
> -DEF_HELPER_0(restore_owb, void)
> -DEF_HELPER_1(movsp, void, i32)
> -DEF_HELPER_1(wsr_lbeg, void, i32)
> -DEF_HELPER_1(wsr_lend, void, i32)
> +DEF_HELPER_2(wsr_windowbase, void, env, i32)
> +DEF_HELPER_4(entry, void, env, i32, i32, i32)
> +DEF_HELPER_2(retw, i32, env, i32)
> +DEF_HELPER_2(rotw, void, env, i32)
> +DEF_HELPER_3(window_check, void, env, i32, i32)
> +DEF_HELPER_1(restore_owb, void, env)
> +DEF_HELPER_2(movsp, void, env, i32)
> +DEF_HELPER_2(wsr_lbeg, void, env, i32)
> +DEF_HELPER_2(wsr_lend, void, env, i32)
> DEF_HELPER_1(simcall, void, env)
> -DEF_HELPER_0(dump_state, void)
> +DEF_HELPER_1(dump_state, void, env)
>
> -DEF_HELPER_2(waiti, void, i32, i32)
> -DEF_HELPER_2(timer_irq, void, i32, i32)
> -DEF_HELPER_1(advance_ccount, void, i32)
> +DEF_HELPER_3(waiti, void, env, i32, i32)
> +DEF_HELPER_3(timer_irq, void, env, i32, i32)
> +DEF_HELPER_2(advance_ccount, void, env, i32)
> DEF_HELPER_1(check_interrupts, void, env)
>
> -DEF_HELPER_1(wsr_rasid, void, i32)
> -DEF_HELPER_FLAGS_2(rtlb0, TCG_CALL_CONST | TCG_CALL_PURE, i32, i32, i32)
> -DEF_HELPER_FLAGS_2(rtlb1, TCG_CALL_CONST | TCG_CALL_PURE, i32, i32, i32)
> -DEF_HELPER_2(itlb, void, i32, i32)
> -DEF_HELPER_2(ptlb, i32, i32, i32)
> -DEF_HELPER_3(wtlb, void, i32, i32, i32)
> +DEF_HELPER_2(wsr_rasid, void, env, i32)
> +DEF_HELPER_FLAGS_3(rtlb0, TCG_CALL_CONST | TCG_CALL_PURE, i32, env, i32, i32)
> +DEF_HELPER_FLAGS_3(rtlb1, TCG_CALL_CONST | TCG_CALL_PURE, i32, env, i32, i32)
> +DEF_HELPER_3(itlb, void, env, i32, i32)
> +DEF_HELPER_3(ptlb, i32, env, i32, i32)
> +DEF_HELPER_4(wtlb, void, env, i32, i32, i32)
>
> -DEF_HELPER_1(wsr_ibreakenable, void, i32)
> -DEF_HELPER_2(wsr_ibreaka, void, i32, i32)
> -DEF_HELPER_2(wsr_dbreaka, void, i32, i32)
> -DEF_HELPER_2(wsr_dbreakc, void, i32, i32)
> +DEF_HELPER_2(wsr_ibreakenable, void, env, i32)
> +DEF_HELPER_3(wsr_ibreaka, void, env, i32, i32)
> +DEF_HELPER_3(wsr_dbreaka, void, env, i32, i32)
> +DEF_HELPER_3(wsr_dbreakc, void, env, i32, i32)
>
> #include "def-helper.h"
> diff --git a/target-xtensa/op_helper.c b/target-xtensa/op_helper.c
> index 41107ff..2659c0e 100644
> --- a/target-xtensa/op_helper.c
> +++ b/target-xtensa/op_helper.c
> @@ -26,12 +26,11 @@
> */
>
> #include "cpu.h"
> -#include "dyngen-exec.h"
> #include "helper.h"
> #include "host-utils.h"
>
> -static void do_unaligned_access(target_ulong addr, int is_write, int is_user,
> - uintptr_t retaddr);
> +static void do_unaligned_access(CPUXtensaState *env,
> + target_ulong addr, int is_write, int is_user, uintptr_t retaddr);
>
> #define ALIGNED_ONLY
> #define MMUSUFFIX _mmu
> @@ -48,7 +47,7 @@ static void do_unaligned_access(target_ulong addr, int
> is_write, int is_user,
> #define SHIFT 3
> #include "softmmu_template.h"
>
> -static void do_restore_state(uintptr_t pc)
> +static void do_restore_state(CPUXtensaState *env, uintptr_t pc)
> {
> TranslationBlock *tb;
>
> @@ -58,44 +57,38 @@ static void do_restore_state(uintptr_t pc)
> }
> }
>
> -static void do_unaligned_access(target_ulong addr, int is_write, int is_user,
> - uintptr_t retaddr)
> +static void do_unaligned_access(CPUXtensaState *env,
> + target_ulong addr, int is_write, int is_user, uintptr_t retaddr)
> {
> if (xtensa_option_enabled(env->config, XTENSA_OPTION_UNALIGNED_EXCEPTION)
> &&
> !xtensa_option_enabled(env->config, XTENSA_OPTION_HW_ALIGNMENT)) {
> - do_restore_state(retaddr);
> - HELPER(exception_cause_vaddr)(
> + do_restore_state(env, retaddr);
> + HELPER(exception_cause_vaddr)(env,
> env->pc, LOAD_STORE_ALIGNMENT_CAUSE, addr);
> }
> }
>
> -void tlb_fill(CPUXtensaState *env1, target_ulong vaddr, int is_write, int
> mmu_idx,
> - uintptr_t retaddr)
> +void tlb_fill(CPUXtensaState *env,
> + target_ulong vaddr, int is_write, int mmu_idx, uintptr_t retaddr)
> {
> - CPUXtensaState *saved_env = env;
> -
> - env = env1;
> - {
> - uint32_t paddr;
> - uint32_t page_size;
> - unsigned access;
> - int ret = xtensa_get_physical_addr(env, true, vaddr, is_write,
> mmu_idx,
> - &paddr, &page_size, &access);
> + uint32_t paddr;
> + uint32_t page_size;
> + unsigned access;
> + int ret = xtensa_get_physical_addr(env, true, vaddr, is_write, mmu_idx,
> + &paddr, &page_size, &access);
>
> - qemu_log("%s(%08x, %d, %d) -> %08x, ret = %d\n", __func__,
> - vaddr, is_write, mmu_idx, paddr, ret);
> + qemu_log("%s(%08x, %d, %d) -> %08x, ret = %d\n", __func__,
> + vaddr, is_write, mmu_idx, paddr, ret);
>
> - if (ret == 0) {
> - tlb_set_page(env,
> - vaddr & TARGET_PAGE_MASK,
> - paddr & TARGET_PAGE_MASK,
> - access, mmu_idx, page_size);
> - } else {
> - do_restore_state(retaddr);
> - HELPER(exception_cause_vaddr)(env->pc, ret, vaddr);
> - }
> + if (ret == 0) {
> + tlb_set_page(env,
> + vaddr & TARGET_PAGE_MASK,
> + paddr & TARGET_PAGE_MASK,
> + access, mmu_idx, page_size);
> + } else {
> + do_restore_state(env, retaddr);
> + HELPER(exception_cause_vaddr)(env, env->pc, ret, vaddr);
> }
> - env = saved_env;
> }
>
> static void tb_invalidate_virtual_addr(CPUXtensaState *env, uint32_t vaddr)
> @@ -110,13 +103,13 @@ static void tb_invalidate_virtual_addr(CPUXtensaState
> *env, uint32_t vaddr)
> }
> }
>
> -void HELPER(exception)(uint32_t excp)
> +void HELPER(exception)(CPUXtensaState *env, uint32_t excp)
> {
> env->exception_index = excp;
> cpu_loop_exit(env);
> }
>
> -void HELPER(exception_cause)(uint32_t pc, uint32_t cause)
> +void HELPER(exception_cause)(CPUXtensaState *env, uint32_t pc, uint32_t
> cause)
> {
> uint32_t vector;
>
> @@ -136,24 +129,24 @@ void HELPER(exception_cause)(uint32_t pc, uint32_t
> cause)
> env->sregs[EXCCAUSE] = cause;
> env->sregs[PS] |= PS_EXCM;
>
> - HELPER(exception)(vector);
> + HELPER(exception)(env, vector);
> }
>
> -void HELPER(exception_cause_vaddr)(uint32_t pc, uint32_t cause, uint32_t
> vaddr)
> +void HELPER(exception_cause_vaddr)(CPUXtensaState *env,
> + uint32_t pc, uint32_t cause, uint32_t vaddr)
> {
> env->sregs[EXCVADDR] = vaddr;
> - HELPER(exception_cause)(pc, cause);
> + HELPER(exception_cause)(env, pc, cause);
> }
>
> -void debug_exception_env(CPUXtensaState *new_env, uint32_t cause)
> +void debug_exception_env(CPUXtensaState *env, uint32_t cause)
> {
> - if (xtensa_get_cintlevel(new_env) < new_env->config->debug_level) {
> - env = new_env;
> - HELPER(debug_exception)(env->pc, cause);
> + if (xtensa_get_cintlevel(env) < env->config->debug_level) {
> + HELPER(debug_exception)(env, env->pc, cause);
> }
> }
>
> -void HELPER(debug_exception)(uint32_t pc, uint32_t cause)
> +void HELPER(debug_exception)(CPUXtensaState *env, uint32_t pc, uint32_t
> cause)
> {
> unsigned level = env->config->debug_level;
>
> @@ -163,7 +156,7 @@ void HELPER(debug_exception)(uint32_t pc, uint32_t cause)
> env->sregs[EPS2 + level - 2] = env->sregs[PS];
> env->sregs[PS] = (env->sregs[PS] & ~PS_INTLEVEL) | PS_EXCM |
> (level << PS_INTLEVEL_SHIFT);
> - HELPER(exception)(EXC_DEBUG);
> + HELPER(exception)(env, EXC_DEBUG);
> }
>
> uint32_t HELPER(nsa)(uint32_t v)
> @@ -232,39 +225,39 @@ void xtensa_sync_phys_from_window(CPUXtensaState *env)
> copy_phys_from_window(env, env->sregs[WINDOW_BASE] * 4, 0, 16);
> }
>
> -static void rotate_window_abs(uint32_t position)
> +static void rotate_window_abs(CPUXtensaState *env, uint32_t position)
> {
> xtensa_sync_phys_from_window(env);
> env->sregs[WINDOW_BASE] = windowbase_bound(position, env);
> xtensa_sync_window_from_phys(env);
> }
>
> -static void rotate_window(uint32_t delta)
> +static void rotate_window(CPUXtensaState *env, uint32_t delta)
> {
> - rotate_window_abs(env->sregs[WINDOW_BASE] + delta);
> + rotate_window_abs(env, env->sregs[WINDOW_BASE] + delta);
> }
>
> -void HELPER(wsr_windowbase)(uint32_t v)
> +void HELPER(wsr_windowbase)(CPUXtensaState *env, uint32_t v)
> {
> - rotate_window_abs(v);
> + rotate_window_abs(env, v);
> }
>
> -void HELPER(entry)(uint32_t pc, uint32_t s, uint32_t imm)
> +void HELPER(entry)(CPUXtensaState *env, uint32_t pc, uint32_t s, uint32_t
> imm)
> {
> int callinc = (env->sregs[PS] & PS_CALLINC) >> PS_CALLINC_SHIFT;
> if (s > 3 || ((env->sregs[PS] & (PS_WOE | PS_EXCM)) ^ PS_WOE) != 0) {
> qemu_log("Illegal entry instruction(pc = %08x), PS = %08x\n",
> pc, env->sregs[PS]);
> - HELPER(exception_cause)(pc, ILLEGAL_INSTRUCTION_CAUSE);
> + HELPER(exception_cause)(env, pc, ILLEGAL_INSTRUCTION_CAUSE);
> } else {
> env->regs[(callinc << 2) | (s & 3)] = env->regs[s] - (imm << 3);
> - rotate_window(callinc);
> + rotate_window(env, callinc);
> env->sregs[WINDOW_START] |=
> windowstart_bit(env->sregs[WINDOW_BASE], env);
> }
> }
>
> -void HELPER(window_check)(uint32_t pc, uint32_t w)
> +void HELPER(window_check)(CPUXtensaState *env, uint32_t pc, uint32_t w)
> {
> uint32_t windowbase = windowbase_bound(env->sregs[WINDOW_BASE], env);
> uint32_t windowstart = env->sregs[WINDOW_START];
> @@ -284,21 +277,21 @@ void HELPER(window_check)(uint32_t pc, uint32_t w)
> }
>
> m = windowbase_bound(windowbase + n, env);
> - rotate_window(n);
> + rotate_window(env, n);
> env->sregs[PS] = (env->sregs[PS] & ~PS_OWB) |
> (windowbase << PS_OWB_SHIFT) | PS_EXCM;
> env->sregs[EPC1] = env->pc = pc;
>
> if (windowstart & windowstart_bit(m + 1, env)) {
> - HELPER(exception)(EXC_WINDOW_OVERFLOW4);
> + HELPER(exception)(env, EXC_WINDOW_OVERFLOW4);
> } else if (windowstart & windowstart_bit(m + 2, env)) {
> - HELPER(exception)(EXC_WINDOW_OVERFLOW8);
> + HELPER(exception)(env, EXC_WINDOW_OVERFLOW8);
> } else {
> - HELPER(exception)(EXC_WINDOW_OVERFLOW12);
> + HELPER(exception)(env, EXC_WINDOW_OVERFLOW12);
> }
> }
>
> -uint32_t HELPER(retw)(uint32_t pc)
> +uint32_t HELPER(retw)(CPUXtensaState *env, uint32_t pc)
> {
> int n = (env->regs[0] >> 30) & 0x3;
> int m = 0;
> @@ -319,13 +312,13 @@ uint32_t HELPER(retw)(uint32_t pc)
> qemu_log("Illegal retw instruction(pc = %08x), "
> "PS = %08x, m = %d, n = %d\n",
> pc, env->sregs[PS], m, n);
> - HELPER(exception_cause)(pc, ILLEGAL_INSTRUCTION_CAUSE);
> + HELPER(exception_cause)(env, pc, ILLEGAL_INSTRUCTION_CAUSE);
> } else {
> int owb = windowbase;
>
> ret_pc = (pc & 0xc0000000) | (env->regs[0] & 0x3fffffff);
>
> - rotate_window(-n);
> + rotate_window(env, -n);
> if (windowstart & windowstart_bit(env->sregs[WINDOW_BASE], env)) {
> env->sregs[WINDOW_START] &= ~windowstart_bit(owb, env);
> } else {
> @@ -335,38 +328,38 @@ uint32_t HELPER(retw)(uint32_t pc)
> env->sregs[EPC1] = env->pc = pc;
>
> if (n == 1) {
> - HELPER(exception)(EXC_WINDOW_UNDERFLOW4);
> + HELPER(exception)(env, EXC_WINDOW_UNDERFLOW4);
> } else if (n == 2) {
> - HELPER(exception)(EXC_WINDOW_UNDERFLOW8);
> + HELPER(exception)(env, EXC_WINDOW_UNDERFLOW8);
> } else if (n == 3) {
> - HELPER(exception)(EXC_WINDOW_UNDERFLOW12);
> + HELPER(exception)(env, EXC_WINDOW_UNDERFLOW12);
> }
> }
> }
> return ret_pc;
> }
>
> -void HELPER(rotw)(uint32_t imm4)
> +void HELPER(rotw)(CPUXtensaState *env, uint32_t imm4)
> {
> - rotate_window(imm4);
> + rotate_window(env, imm4);
> }
>
> -void HELPER(restore_owb)(void)
> +void HELPER(restore_owb)(CPUXtensaState *env)
> {
> - rotate_window_abs((env->sregs[PS] & PS_OWB) >> PS_OWB_SHIFT);
> + rotate_window_abs(env, (env->sregs[PS] & PS_OWB) >> PS_OWB_SHIFT);
> }
>
> -void HELPER(movsp)(uint32_t pc)
> +void HELPER(movsp)(CPUXtensaState *env, uint32_t pc)
> {
> if ((env->sregs[WINDOW_START] &
> (windowstart_bit(env->sregs[WINDOW_BASE] - 3, env) |
> windowstart_bit(env->sregs[WINDOW_BASE] - 2, env) |
> windowstart_bit(env->sregs[WINDOW_BASE] - 1, env))) == 0) {
> - HELPER(exception_cause)(pc, ALLOCA_CAUSE);
> + HELPER(exception_cause)(env, pc, ALLOCA_CAUSE);
> }
> }
>
> -void HELPER(wsr_lbeg)(uint32_t v)
> +void HELPER(wsr_lbeg)(CPUXtensaState *env, uint32_t v)
> {
> if (env->sregs[LBEG] != v) {
> tb_invalidate_virtual_addr(env, env->sregs[LEND] - 1);
> @@ -374,7 +367,7 @@ void HELPER(wsr_lbeg)(uint32_t v)
> }
> }
>
> -void HELPER(wsr_lend)(uint32_t v)
> +void HELPER(wsr_lend)(CPUXtensaState *env, uint32_t v)
> {
> if (env->sregs[LEND] != v) {
> tb_invalidate_virtual_addr(env, env->sregs[LEND] - 1);
> @@ -383,12 +376,12 @@ void HELPER(wsr_lend)(uint32_t v)
> }
> }
>
> -void HELPER(dump_state)(void)
> +void HELPER(dump_state)(CPUXtensaState *env)
> {
> cpu_dump_state(env, stderr, fprintf, 0);
> }
>
> -void HELPER(waiti)(uint32_t pc, uint32_t intlevel)
> +void HELPER(waiti)(CPUXtensaState *env, uint32_t pc, uint32_t intlevel)
> {
> env->pc = pc;
> env->sregs[PS] = (env->sregs[PS] & ~PS_INTLEVEL) |
> @@ -404,15 +397,15 @@ void HELPER(waiti)(uint32_t pc, uint32_t intlevel)
> if (xtensa_option_enabled(env->config, XTENSA_OPTION_TIMER_INTERRUPT)) {
> xtensa_rearm_ccompare_timer(env);
> }
> - HELPER(exception)(EXCP_HLT);
> + HELPER(exception)(env, EXCP_HLT);
> }
>
> -void HELPER(timer_irq)(uint32_t id, uint32_t active)
> +void HELPER(timer_irq)(CPUXtensaState *env, uint32_t id, uint32_t active)
> {
> xtensa_timer_irq(env, id, active);
> }
>
> -void HELPER(advance_ccount)(uint32_t d)
> +void HELPER(advance_ccount)(CPUXtensaState *env, uint32_t d)
> {
> xtensa_advance_ccount(env, d);
> }
> @@ -422,7 +415,7 @@ void HELPER(check_interrupts)(CPUXtensaState *env)
> check_interrupts(env);
> }
>
> -void HELPER(wsr_rasid)(uint32_t v)
> +void HELPER(wsr_rasid)(CPUXtensaState *env, uint32_t v)
> {
> v = (v & 0xffffff00) | 0x1;
> if (v != env->sregs[RASID]) {
> @@ -574,7 +567,7 @@ void split_tlb_entry_spec_way(const CPUXtensaState *env,
> uint32_t v, bool dtlb,
> * Split TLB address into TLB way, entry index and VPN (with index).
> * See ISA, 4.6.5.5 - 4.6.5.8 for the TLB addressing format
> */
> -static void split_tlb_entry_spec(uint32_t v, bool dtlb,
> +static void split_tlb_entry_spec(CPUXtensaState *env, uint32_t v, bool dtlb,
> uint32_t *vpn, uint32_t *wi, uint32_t *ei)
> {
> if (xtensa_option_enabled(env->config, XTENSA_OPTION_MMU)) {
> @@ -587,41 +580,42 @@ static void split_tlb_entry_spec(uint32_t v, bool dtlb,
> }
> }
>
> -static xtensa_tlb_entry *get_tlb_entry(uint32_t v, bool dtlb, uint32_t *pwi)
> +static xtensa_tlb_entry *get_tlb_entry(CPUXtensaState *env,
> + uint32_t v, bool dtlb, uint32_t *pwi)
> {
> uint32_t vpn;
> uint32_t wi;
> uint32_t ei;
>
> - split_tlb_entry_spec(v, dtlb, &vpn, &wi, &ei);
> + split_tlb_entry_spec(env, v, dtlb, &vpn, &wi, &ei);
> if (pwi) {
> *pwi = wi;
> }
> return xtensa_tlb_get_entry(env, dtlb, wi, ei);
> }
>
> -uint32_t HELPER(rtlb0)(uint32_t v, uint32_t dtlb)
> +uint32_t HELPER(rtlb0)(CPUXtensaState *env, uint32_t v, uint32_t dtlb)
> {
> if (xtensa_option_enabled(env->config, XTENSA_OPTION_MMU)) {
> uint32_t wi;
> - const xtensa_tlb_entry *entry = get_tlb_entry(v, dtlb, &wi);
> + const xtensa_tlb_entry *entry = get_tlb_entry(env, v, dtlb, &wi);
> return (entry->vaddr & get_vpn_mask(env, dtlb, wi)) | entry->asid;
> } else {
> return v & REGION_PAGE_MASK;
> }
> }
>
> -uint32_t HELPER(rtlb1)(uint32_t v, uint32_t dtlb)
> +uint32_t HELPER(rtlb1)(CPUXtensaState *env, uint32_t v, uint32_t dtlb)
> {
> - const xtensa_tlb_entry *entry = get_tlb_entry(v, dtlb, NULL);
> + const xtensa_tlb_entry *entry = get_tlb_entry(env, v, dtlb, NULL);
> return entry->paddr | entry->attr;
> }
>
> -void HELPER(itlb)(uint32_t v, uint32_t dtlb)
> +void HELPER(itlb)(CPUXtensaState *env, uint32_t v, uint32_t dtlb)
> {
> if (xtensa_option_enabled(env->config, XTENSA_OPTION_MMU)) {
> uint32_t wi;
> - xtensa_tlb_entry *entry = get_tlb_entry(v, dtlb, &wi);
> + xtensa_tlb_entry *entry = get_tlb_entry(env, v, dtlb, &wi);
> if (entry->variable && entry->asid) {
> tlb_flush_page(env, entry->vaddr);
> entry->asid = 0;
> @@ -629,7 +623,7 @@ void HELPER(itlb)(uint32_t v, uint32_t dtlb)
> }
> }
>
> -uint32_t HELPER(ptlb)(uint32_t v, uint32_t dtlb)
> +uint32_t HELPER(ptlb)(CPUXtensaState *env, uint32_t v, uint32_t dtlb)
> {
> if (xtensa_option_enabled(env->config, XTENSA_OPTION_MMU)) {
> uint32_t wi;
> @@ -646,7 +640,7 @@ uint32_t HELPER(ptlb)(uint32_t v, uint32_t dtlb)
>
> case INST_TLB_MULTI_HIT_CAUSE:
> case LOAD_STORE_TLB_MULTI_HIT_CAUSE:
> - HELPER(exception_cause_vaddr)(env->pc, res, v);
> + HELPER(exception_cause_vaddr)(env, env->pc, res, v);
> break;
> }
> return 0;
> @@ -691,17 +685,17 @@ void xtensa_tlb_set_entry(CPUXtensaState *env, bool
> dtlb,
> }
> }
>
> -void HELPER(wtlb)(uint32_t p, uint32_t v, uint32_t dtlb)
> +void HELPER(wtlb)(CPUXtensaState *env, uint32_t p, uint32_t v, uint32_t dtlb)
> {
> uint32_t vpn;
> uint32_t wi;
> uint32_t ei;
> - split_tlb_entry_spec(v, dtlb, &vpn, &wi, &ei);
> + split_tlb_entry_spec(env, v, dtlb, &vpn, &wi, &ei);
> xtensa_tlb_set_entry(env, dtlb, wi, ei, vpn, p);
> }
>
>
> -void HELPER(wsr_ibreakenable)(uint32_t v)
> +void HELPER(wsr_ibreakenable)(CPUXtensaState *env, uint32_t v)
> {
> uint32_t change = v ^ env->sregs[IBREAKENABLE];
> unsigned i;
> @@ -714,7 +708,7 @@ void HELPER(wsr_ibreakenable)(uint32_t v)
> env->sregs[IBREAKENABLE] = v & ((1 << env->config->nibreak) - 1);
> }
>
> -void HELPER(wsr_ibreaka)(uint32_t i, uint32_t v)
> +void HELPER(wsr_ibreaka)(CPUXtensaState *env, uint32_t i, uint32_t v)
> {
> if (env->sregs[IBREAKENABLE] & (1 << i) && env->sregs[IBREAKA + i] != v) {
> tb_invalidate_virtual_addr(env, env->sregs[IBREAKA + i]);
> @@ -723,7 +717,8 @@ void HELPER(wsr_ibreaka)(uint32_t i, uint32_t v)
> env->sregs[IBREAKA + i] = v;
> }
>
> -static void set_dbreak(unsigned i, uint32_t dbreaka, uint32_t dbreakc)
> +static void set_dbreak(CPUXtensaState *env, unsigned i, uint32_t dbreaka,
> + uint32_t dbreakc)
> {
> int flags = BP_CPU | BP_STOP_BEFORE_ACCESS;
> uint32_t mask = dbreakc | ~DBREAKC_MASK;
> @@ -751,22 +746,22 @@ static void set_dbreak(unsigned i, uint32_t dbreaka,
> uint32_t dbreakc)
> }
> }
>
> -void HELPER(wsr_dbreaka)(uint32_t i, uint32_t v)
> +void HELPER(wsr_dbreaka)(CPUXtensaState *env, uint32_t i, uint32_t v)
> {
> uint32_t dbreakc = env->sregs[DBREAKC + i];
>
> if ((dbreakc & DBREAKC_SB_LB) &&
> env->sregs[DBREAKA + i] != v) {
> - set_dbreak(i, v, dbreakc);
> + set_dbreak(env, i, v, dbreakc);
> }
> env->sregs[DBREAKA + i] = v;
> }
>
> -void HELPER(wsr_dbreakc)(uint32_t i, uint32_t v)
> +void HELPER(wsr_dbreakc)(CPUXtensaState *env, uint32_t i, uint32_t v)
> {
> if ((env->sregs[DBREAKC + i] ^ v) & (DBREAKC_SB_LB | DBREAKC_MASK)) {
> if (v & DBREAKC_SB_LB) {
> - set_dbreak(i, env->sregs[DBREAKA + i], v);
> + set_dbreak(env, i, env->sregs[DBREAKA + i], v);
> } else {
> if (env->cpu_watchpoint[i]) {
> cpu_watchpoint_remove_by_ref(env, env->cpu_watchpoint[i]);
> diff --git a/target-xtensa/translate.c b/target-xtensa/translate.c
> index a542a31..b883e6b 100644
> --- a/target-xtensa/translate.c
> +++ b/target-xtensa/translate.c
> @@ -254,7 +254,7 @@ static void gen_advance_ccount(DisasContext *dc)
> if (dc->ccount_delta > 0) {
> TCGv_i32 tmp = tcg_const_i32(dc->ccount_delta);
> dc->ccount_delta = 0;
> - gen_helper_advance_ccount(tmp);
> + gen_helper_advance_ccount(cpu_env, tmp);
> tcg_temp_free(tmp);
> }
> }
> @@ -268,7 +268,7 @@ static void gen_exception(DisasContext *dc, int excp)
> {
> TCGv_i32 tmp = tcg_const_i32(excp);
> gen_advance_ccount(dc);
> - gen_helper_exception(tmp);
> + gen_helper_exception(cpu_env, tmp);
> tcg_temp_free(tmp);
> }
>
> @@ -277,7 +277,7 @@ static void gen_exception_cause(DisasContext *dc,
> uint32_t cause)
> TCGv_i32 tpc = tcg_const_i32(dc->pc);
> TCGv_i32 tcause = tcg_const_i32(cause);
> gen_advance_ccount(dc);
> - gen_helper_exception_cause(tpc, tcause);
> + gen_helper_exception_cause(cpu_env, tpc, tcause);
> tcg_temp_free(tpc);
> tcg_temp_free(tcause);
> if (cause == ILLEGAL_INSTRUCTION_CAUSE ||
> @@ -292,7 +292,7 @@ static void gen_exception_cause_vaddr(DisasContext *dc,
> uint32_t cause,
> TCGv_i32 tpc = tcg_const_i32(dc->pc);
> TCGv_i32 tcause = tcg_const_i32(cause);
> gen_advance_ccount(dc);
> - gen_helper_exception_cause_vaddr(tpc, tcause, vaddr);
> + gen_helper_exception_cause_vaddr(cpu_env, tpc, tcause, vaddr);
> tcg_temp_free(tpc);
> tcg_temp_free(tcause);
> }
> @@ -302,7 +302,7 @@ static void gen_debug_exception(DisasContext *dc,
> uint32_t cause)
> TCGv_i32 tpc = tcg_const_i32(dc->pc);
> TCGv_i32 tcause = tcg_const_i32(cause);
> gen_advance_ccount(dc);
> - gen_helper_debug_exception(tpc, tcause);
> + gen_helper_debug_exception(cpu_env, tpc, tcause);
> tcg_temp_free(tpc);
> tcg_temp_free(tcause);
> if (cause & (DEBUGCAUSE_IB | DEBUGCAUSE_BI | DEBUGCAUSE_BN)) {
> @@ -460,13 +460,13 @@ static void gen_rsr(DisasContext *dc, TCGv_i32 d,
> uint32_t sr)
>
> static void gen_wsr_lbeg(DisasContext *dc, uint32_t sr, TCGv_i32 s)
> {
> - gen_helper_wsr_lbeg(s);
> + gen_helper_wsr_lbeg(cpu_env, s);
> gen_jumpi_check_loop_end(dc, 0);
> }
>
> static void gen_wsr_lend(DisasContext *dc, uint32_t sr, TCGv_i32 s)
> {
> - gen_helper_wsr_lend(s);
> + gen_helper_wsr_lend(cpu_env, s);
> gen_jumpi_check_loop_end(dc, 0);
> }
>
> @@ -499,7 +499,7 @@ static void gen_wsr_acchi(DisasContext *dc, uint32_t sr,
> TCGv_i32 s)
>
> static void gen_wsr_windowbase(DisasContext *dc, uint32_t sr, TCGv_i32 v)
> {
> - gen_helper_wsr_windowbase(v);
> + gen_helper_wsr_windowbase(cpu_env, v);
> reset_used_window(dc);
> }
>
> @@ -516,7 +516,7 @@ static void gen_wsr_ptevaddr(DisasContext *dc, uint32_t
> sr, TCGv_i32 v)
>
> static void gen_wsr_rasid(DisasContext *dc, uint32_t sr, TCGv_i32 v)
> {
> - gen_helper_wsr_rasid(v);
> + gen_helper_wsr_rasid(cpu_env, v);
> /* This can change tb->flags, so exit tb */
> gen_jumpi_check_loop_end(dc, -1);
> }
> @@ -528,7 +528,7 @@ static void gen_wsr_tlbcfg(DisasContext *dc, uint32_t sr,
> TCGv_i32 v)
>
> static void gen_wsr_ibreakenable(DisasContext *dc, uint32_t sr, TCGv_i32 v)
> {
> - gen_helper_wsr_ibreakenable(v);
> + gen_helper_wsr_ibreakenable(cpu_env, v);
> gen_jumpi_check_loop_end(dc, 0);
> }
>
> @@ -538,7 +538,7 @@ static void gen_wsr_ibreaka(DisasContext *dc, uint32_t
> sr, TCGv_i32 v)
>
> if (id < dc->config->nibreak) {
> TCGv_i32 tmp = tcg_const_i32(id);
> - gen_helper_wsr_ibreaka(tmp, v);
> + gen_helper_wsr_ibreaka(cpu_env, tmp, v);
> tcg_temp_free(tmp);
> gen_jumpi_check_loop_end(dc, 0);
> }
> @@ -550,7 +550,7 @@ static void gen_wsr_dbreaka(DisasContext *dc, uint32_t
> sr, TCGv_i32 v)
>
> if (id < dc->config->ndbreak) {
> TCGv_i32 tmp = tcg_const_i32(id);
> - gen_helper_wsr_dbreaka(tmp, v);
> + gen_helper_wsr_dbreaka(cpu_env, tmp, v);
> tcg_temp_free(tmp);
> }
> }
> @@ -561,7 +561,7 @@ static void gen_wsr_dbreakc(DisasContext *dc, uint32_t
> sr, TCGv_i32 v)
>
> if (id < dc->config->ndbreak) {
> TCGv_i32 tmp = tcg_const_i32(id);
> - gen_helper_wsr_dbreakc(tmp, v);
> + gen_helper_wsr_dbreakc(cpu_env, tmp, v);
> tcg_temp_free(tmp);
> }
> }
> @@ -714,7 +714,7 @@ static void gen_waiti(DisasContext *dc, uint32_t imm4)
> TCGv_i32 pc = tcg_const_i32(dc->next_pc);
> TCGv_i32 intlevel = tcg_const_i32(imm4);
> gen_advance_ccount(dc);
> - gen_helper_waiti(pc, intlevel);
> + gen_helper_waiti(cpu_env, pc, intlevel);
> tcg_temp_free(pc);
> tcg_temp_free(intlevel);
> }
> @@ -731,7 +731,7 @@ static void gen_window_check1(DisasContext *dc, unsigned
> r1)
>
> dc->used_window = r1 / 4;
> gen_advance_ccount(dc);
> - gen_helper_window_check(pc, w);
> + gen_helper_window_check(cpu_env, pc, w);
>
> tcg_temp_free(w);
> tcg_temp_free(pc);
> @@ -851,8 +851,8 @@ static void disas_xtensa_insn(DisasContext *dc)
>
> #define RSR_SR (b1)
>
> - uint8_t b0 = ldub_code(dc->pc);
> - uint8_t b1 = ldub_code(dc->pc + 1);
> + uint8_t b0 = cpu_ldub_code(cpu_single_env, dc->pc);
> + uint8_t b1 = cpu_ldub_code(cpu_single_env, dc->pc + 1);
> uint8_t b2 = 0;
>
> static const uint32_t B4CONST[] = {
> @@ -868,7 +868,7 @@ static void disas_xtensa_insn(DisasContext *dc)
> HAS_OPTION(XTENSA_OPTION_CODE_DENSITY);
> } else {
> dc->next_pc = dc->pc + 3;
> - b2 = ldub_code(dc->pc + 2);
> + b2 = cpu_ldub_code(cpu_single_env, dc->pc + 2);
> }
>
> switch (OP0) {
> @@ -905,7 +905,7 @@ static void disas_xtensa_insn(DisasContext *dc)
> {
> TCGv_i32 tmp = tcg_const_i32(dc->pc);
> gen_advance_ccount(dc);
> - gen_helper_retw(tmp, tmp);
> + gen_helper_retw(tmp, cpu_env, tmp);
> gen_jump(dc, tmp);
> tcg_temp_free(tmp);
> }
> @@ -953,7 +953,7 @@ static void disas_xtensa_insn(DisasContext *dc)
> {
> TCGv_i32 pc = tcg_const_i32(dc->pc);
> gen_advance_ccount(dc);
> - gen_helper_movsp(pc);
> + gen_helper_movsp(cpu_env, pc);
> tcg_gen_mov_i32(cpu_R[RRR_T], cpu_R[RRR_S]);
> tcg_temp_free(pc);
> }
> @@ -1033,7 +1033,7 @@ static void disas_xtensa_insn(DisasContext *dc)
> cpu_SR[WINDOW_START], tmp);
> }
>
> - gen_helper_restore_owb();
> + gen_helper_restore_owb(cpu_env);
> gen_helper_check_interrupts(cpu_env);
> gen_jump(dc, cpu_SR[EPC1]);
>
> @@ -1221,7 +1221,7 @@ static void disas_xtensa_insn(DisasContext *dc)
> {
> TCGv_i32 tmp = tcg_const_i32(
> RRR_T | ((RRR_T & 8) ? 0xfffffff0 : 0));
> - gen_helper_rotw(tmp);
> + gen_helper_rotw(cpu_env, tmp);
> tcg_temp_free(tmp);
> reset_used_window(dc);
> }
> @@ -1257,28 +1257,32 @@ static void disas_xtensa_insn(DisasContext *dc)
>
> switch (RRR_R & 7) {
> case 3: /*RITLB0*/ /*RDTLB0*/
> - gen_helper_rtlb0(cpu_R[RRR_T], cpu_R[RRR_S], dtlb);
> + gen_helper_rtlb0(cpu_R[RRR_T],
> + cpu_env, cpu_R[RRR_S], dtlb);
> break;
>
> case 4: /*IITLB*/ /*IDTLB*/
> - gen_helper_itlb(cpu_R[RRR_S], dtlb);
> + gen_helper_itlb(cpu_env, cpu_R[RRR_S], dtlb);
> /* This could change memory mapping, so exit tb */
> gen_jumpi_check_loop_end(dc, -1);
> break;
>
> case 5: /*PITLB*/ /*PDTLB*/
> tcg_gen_movi_i32(cpu_pc, dc->pc);
> - gen_helper_ptlb(cpu_R[RRR_T], cpu_R[RRR_S], dtlb);
> + gen_helper_ptlb(cpu_R[RRR_T],
> + cpu_env, cpu_R[RRR_S], dtlb);
> break;
>
> case 6: /*WITLB*/ /*WDTLB*/
> - gen_helper_wtlb(cpu_R[RRR_T], cpu_R[RRR_S], dtlb);
> + gen_helper_wtlb(
> + cpu_env, cpu_R[RRR_T], cpu_R[RRR_S], dtlb);
> /* This could change memory mapping, so exit tb */
> gen_jumpi_check_loop_end(dc, -1);
> break;
>
> case 7: /*RITLB1*/ /*RDTLB1*/
> - gen_helper_rtlb1(cpu_R[RRR_T], cpu_R[RRR_S], dtlb);
> + gen_helper_rtlb1(cpu_R[RRR_T],
> + cpu_env, cpu_R[RRR_S], dtlb);
> break;
>
> default:
> @@ -2246,7 +2250,7 @@ static void disas_xtensa_insn(DisasContext *dc)
> TCGv_i32 s = tcg_const_i32(BRI12_S);
> TCGv_i32 imm = tcg_const_i32(BRI12_IMM12);
> gen_advance_ccount(dc);
> - gen_helper_entry(pc, s, imm);
> + gen_helper_entry(cpu_env, pc, s, imm);
> tcg_temp_free(imm);
> tcg_temp_free(s);
> tcg_temp_free(pc);
> @@ -2280,7 +2284,7 @@ static void disas_xtensa_insn(DisasContext *dc)
>
> tcg_gen_subi_i32(cpu_SR[LCOUNT], cpu_R[RRI8_S], 1);
> tcg_gen_movi_i32(cpu_SR[LBEG], dc->next_pc);
> - gen_helper_wsr_lend(tmp);
> + gen_helper_wsr_lend(cpu_env, tmp);
> tcg_temp_free(tmp);
>
> if (BRI8_R > 8) {
> @@ -2449,7 +2453,7 @@ static void disas_xtensa_insn(DisasContext *dc)
> {
> TCGv_i32 tmp = tcg_const_i32(dc->pc);
> gen_advance_ccount(dc);
> - gen_helper_retw(tmp, tmp);
> + gen_helper_retw(tmp, cpu_env, tmp);
> gen_jump(dc, tmp);
> tcg_temp_free(tmp);
> }
> --
> 1.7.7.6
>
>