diff mbox series

[v3,26/39] target/sparc: Use env_cpu, env_archcpu

Message ID 20190508000641.19090-27-richard.henderson@linaro.org
State Superseded
Headers show
Series tcg: Move the softmmu tlb to CPUNegativeOffsetState | expand

Commit Message

Richard Henderson May 8, 2019, 12:06 a.m. UTC
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

---
 target/sparc/cpu.h          |  5 -----
 bsd-user/main.c             |  2 +-
 hw/sparc/leon3.c            |  4 ++--
 hw/sparc/sun4m.c            |  4 ++--
 hw/sparc64/sparc64.c        |  2 +-
 linux-user/sparc/cpu_loop.c |  2 +-
 target/sparc/fop_helper.c   |  2 +-
 target/sparc/helper.c       |  8 ++++----
 target/sparc/ldst_helper.c  | 33 +++++++++++++++------------------
 target/sparc/mmu_helper.c   | 10 +++++-----
 10 files changed, 32 insertions(+), 40 deletions(-)

-- 
2.17.1

Comments

Alistair Francis May 9, 2019, 8:15 p.m. UTC | #1
On Tue, May 7, 2019 at 5:25 PM Richard Henderson
<richard.henderson@linaro.org> wrote:
>

> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Reviewed-by: Alistair Francis <alistair.francis@wdc.com>


Alistair

> ---

>  target/sparc/cpu.h          |  5 -----

>  bsd-user/main.c             |  2 +-

>  hw/sparc/leon3.c            |  4 ++--

>  hw/sparc/sun4m.c            |  4 ++--

>  hw/sparc64/sparc64.c        |  2 +-

>  linux-user/sparc/cpu_loop.c |  2 +-

>  target/sparc/fop_helper.c   |  2 +-

>  target/sparc/helper.c       |  8 ++++----

>  target/sparc/ldst_helper.c  | 33 +++++++++++++++------------------

>  target/sparc/mmu_helper.c   | 10 +++++-----

>  10 files changed, 32 insertions(+), 40 deletions(-)

>

> diff --git a/target/sparc/cpu.h b/target/sparc/cpu.h

> index 5a10680d8f..1442232cef 100644

> --- a/target/sparc/cpu.h

> +++ b/target/sparc/cpu.h

> @@ -532,11 +532,6 @@ struct SPARCCPU {

>      CPUSPARCState env;

>  };

>

> -static inline SPARCCPU *sparc_env_get_cpu(CPUSPARCState *env)

> -{

> -    return container_of(env, SPARCCPU, env);

> -}

> -

>  #define ENV_OFFSET offsetof(SPARCCPU, env)

>

>  #ifndef CONFIG_USER_ONLY

> diff --git a/bsd-user/main.c b/bsd-user/main.c

> index bfdcae4269..f6f76f1a64 100644

> --- a/bsd-user/main.c

> +++ b/bsd-user/main.c

> @@ -486,7 +486,7 @@ static void flush_windows(CPUSPARCState *env)

>

>  void cpu_loop(CPUSPARCState *env)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>      int trapnr, ret, syscall_nr;

>      //target_siginfo_t info;

>

> diff --git a/hw/sparc/leon3.c b/hw/sparc/leon3.c

> index 774639af33..ef74bc81c2 100644

> --- a/hw/sparc/leon3.c

> +++ b/hw/sparc/leon3.c

> @@ -91,7 +91,7 @@ static void leon3_set_pil_in(void *opaque, uint32_t pil_in)

>

>                  env->interrupt_index = TT_EXTINT | i;

>                  if (old_interrupt != env->interrupt_index) {

> -                    cs = CPU(sparc_env_get_cpu(env));

> +                    cs = env_cpu(env);

>                      trace_leon3_set_irq(i);

>                      cpu_interrupt(cs, CPU_INTERRUPT_HARD);

>                  }

> @@ -99,7 +99,7 @@ static void leon3_set_pil_in(void *opaque, uint32_t pil_in)

>              }

>          }

>      } else if (!env->pil_in && (env->interrupt_index & ~15) == TT_EXTINT) {

> -        cs = CPU(sparc_env_get_cpu(env));

> +        cs = env_cpu(env);

>          trace_leon3_reset_irq(env->interrupt_index & 15);

>          env->interrupt_index = 0;

>          cpu_reset_interrupt(cs, CPU_INTERRUPT_HARD);

> diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c

> index ca1e3825d5..a87bef6d4f 100644

> --- a/hw/sparc/sun4m.c

> +++ b/hw/sparc/sun4m.c

> @@ -147,7 +147,7 @@ void cpu_check_irqs(CPUSPARCState *env)

>

>                  env->interrupt_index = TT_EXTINT | i;

>                  if (old_interrupt != env->interrupt_index) {

> -                    cs = CPU(sparc_env_get_cpu(env));

> +                    cs = env_cpu(env);

>                      trace_sun4m_cpu_interrupt(i);

>                      cpu_interrupt(cs, CPU_INTERRUPT_HARD);

>                  }

> @@ -155,7 +155,7 @@ void cpu_check_irqs(CPUSPARCState *env)

>              }

>          }

>      } else if (!env->pil_in && (env->interrupt_index & ~15) == TT_EXTINT) {

> -        cs = CPU(sparc_env_get_cpu(env));

> +        cs = env_cpu(env);

>          trace_sun4m_cpu_reset_interrupt(env->interrupt_index & 15);

>          env->interrupt_index = 0;

>          cpu_reset_interrupt(cs, CPU_INTERRUPT_HARD);

> diff --git a/hw/sparc64/sparc64.c b/hw/sparc64/sparc64.c

> index 408388945e..689801f37d 100644

> --- a/hw/sparc64/sparc64.c

> +++ b/hw/sparc64/sparc64.c

> @@ -46,7 +46,7 @@ void cpu_check_irqs(CPUSPARCState *env)

>      if (env->ivec_status & 0x20) {

>          return;

>      }

> -    cs = CPU(sparc_env_get_cpu(env));

> +    cs = env_cpu(env);

>      /* check if TM or SM in SOFTINT are set

>         setting these also causes interrupt 14 */

>      if (env->softint & (SOFTINT_TIMER | SOFTINT_STIMER)) {

> diff --git a/linux-user/sparc/cpu_loop.c b/linux-user/sparc/cpu_loop.c

> index 9e357229c0..d85359037c 100644

> --- a/linux-user/sparc/cpu_loop.c

> +++ b/linux-user/sparc/cpu_loop.c

> @@ -145,7 +145,7 @@ static void flush_windows(CPUSPARCState *env)

>

>  void cpu_loop (CPUSPARCState *env)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>      int trapnr;

>      abi_long ret;

>      target_siginfo_t info;

> diff --git a/target/sparc/fop_helper.c b/target/sparc/fop_helper.c

> index b6642fd1d7..9eb9b75718 100644

> --- a/target/sparc/fop_helper.c

> +++ b/target/sparc/fop_helper.c

> @@ -53,7 +53,7 @@ static target_ulong do_check_ieee_exceptions(CPUSPARCState *env, uintptr_t ra)

>          }

>

>          if ((fsr & FSR_CEXC_MASK) & ((fsr & FSR_TEM_MASK) >> 23)) {

> -            CPUState *cs = CPU(sparc_env_get_cpu(env));

> +            CPUState *cs = env_cpu(env);

>

>              /* Unmasked exception, generate a trap.  Note that while

>                 the helper is marked as NO_WG, we can get away with

> diff --git a/target/sparc/helper.c b/target/sparc/helper.c

> index 46232788c8..1a52061fbf 100644

> --- a/target/sparc/helper.c

> +++ b/target/sparc/helper.c

> @@ -26,7 +26,7 @@

>

>  void cpu_raise_exception_ra(CPUSPARCState *env, int tt, uintptr_t ra)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>

>      cs->exception_index = tt;

>      cpu_loop_exit_restore(cs, ra);

> @@ -34,7 +34,7 @@ void cpu_raise_exception_ra(CPUSPARCState *env, int tt, uintptr_t ra)

>

>  void helper_raise_exception(CPUSPARCState *env, int tt)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>

>      cs->exception_index = tt;

>      cpu_loop_exit(cs);

> @@ -42,7 +42,7 @@ void helper_raise_exception(CPUSPARCState *env, int tt)

>

>  void helper_debug(CPUSPARCState *env)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>

>      cs->exception_index = EXCP_DEBUG;

>      cpu_loop_exit(cs);

> @@ -243,7 +243,7 @@ target_ulong helper_tsubcctv(CPUSPARCState *env, target_ulong src1,

>  #ifndef TARGET_SPARC64

>  void helper_power_down(CPUSPARCState *env)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>

>      cs->halted = 1;

>      cs->exception_index = EXCP_HLT;

> diff --git a/target/sparc/ldst_helper.c b/target/sparc/ldst_helper.c

> index a7fcb84ac0..a2d033a61f 100644

> --- a/target/sparc/ldst_helper.c

> +++ b/target/sparc/ldst_helper.c

> @@ -122,13 +122,13 @@ static uint64_t ultrasparc_tag_target(uint64_t tag_access_register)

>

>  static void replace_tlb_entry(SparcTLBEntry *tlb,

>                                uint64_t tlb_tag, uint64_t tlb_tte,

> -                              CPUSPARCState *env1)

> +                              CPUSPARCState *env)

>  {

>      target_ulong mask, size, va, offset;

>

>      /* flush page range if translation is valid */

>      if (TTE_IS_VALID(tlb->tte)) {

> -        CPUState *cs = CPU(sparc_env_get_cpu(env1));

> +        CPUState *cs = env_cpu(env);

>

>          size = 8192ULL << 3 * TTE_PGSIZE(tlb->tte);

>          mask = 1ULL + ~size;

> @@ -499,7 +499,7 @@ uint64_t helper_ld_asi(CPUSPARCState *env, target_ulong addr,

>  {

>      int size = 1 << (memop & MO_SIZE);

>      int sign = memop & MO_SIGN;

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>      uint64_t ret = 0;

>  #if defined(DEBUG_MXCC) || defined(DEBUG_ASI)

>      uint32_t last_addr = addr;

> @@ -725,8 +725,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,

>                     int asi, uint32_t memop)

>  {

>      int size = 1 << (memop & MO_SIZE);

> -    SPARCCPU *cpu = sparc_env_get_cpu(env);

> -    CPUState *cs = CPU(cpu);

> +    CPUState *cs = env_cpu(env);

>

>      do_check_align(env, addr, size - 1, GETPC());

>      switch (asi) {

> @@ -874,13 +873,13 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,

>              DPRINTF_MMU("mmu flush level %d\n", mmulev);

>              switch (mmulev) {

>              case 0: /* flush page */

> -                tlb_flush_page(CPU(cpu), addr & 0xfffff000);

> +                tlb_flush_page(cs, addr & 0xfffff000);

>                  break;

>              case 1: /* flush segment (256k) */

>              case 2: /* flush region (16M) */

>              case 3: /* flush context (4G) */

>              case 4: /* flush entire */

> -                tlb_flush(CPU(cpu));

> +                tlb_flush(cs);

>                  break;

>              default:

>                  break;

> @@ -905,7 +904,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,

>                     are invalid in normal mode.  */

>                  if ((oldreg ^ env->mmuregs[reg])

>                      & (MMU_NF | env->def.mmu_bm)) {

> -                    tlb_flush(CPU(cpu));

> +                    tlb_flush(cs);

>                  }

>                  break;

>              case 1: /* Context Table Pointer Register */

> @@ -916,7 +915,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,

>                  if (oldreg != env->mmuregs[reg]) {

>                      /* we flush when the MMU context changes because

>                         QEMU has no MMU context support */

> -                    tlb_flush(CPU(cpu));

> +                    tlb_flush(cs);

>                  }

>                  break;

>              case 3: /* Synchronous Fault Status Register with Clear */

> @@ -1027,8 +1026,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,

>      case ASI_USERTXT: /* User code access, XXX */

>      case ASI_KERNELTXT: /* Supervisor code access, XXX */

>      default:

> -        cpu_unassigned_access(CPU(sparc_env_get_cpu(env)),

> -                              addr, true, false, asi, size);

> +        cpu_unassigned_access(cs, addr, true, false, asi, size);

>          break;

>

>      case ASI_USERDATA: /* User data access */

> @@ -1175,7 +1173,7 @@ uint64_t helper_ld_asi(CPUSPARCState *env, target_ulong addr,

>  {

>      int size = 1 << (memop & MO_SIZE);

>      int sign = memop & MO_SIGN;

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>      uint64_t ret = 0;

>  #if defined(DEBUG_ASI)

>      target_ulong last_addr = addr;

> @@ -1481,8 +1479,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val,

>                     int asi, uint32_t memop)

>  {

>      int size = 1 << (memop & MO_SIZE);

> -    SPARCCPU *cpu = sparc_env_get_cpu(env);

> -    CPUState *cs = CPU(cpu);

> +    CPUState *cs = env_cpu(env);

>

>  #ifdef DEBUG_ASI

>      dump_asi("write", addr, asi, size, val);

> @@ -1686,13 +1683,13 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val,

>                  env->dmmu.mmu_primary_context = val;

>                  /* can be optimized to only flush MMU_USER_IDX

>                     and MMU_KERNEL_IDX entries */

> -                tlb_flush(CPU(cpu));

> +                tlb_flush(cs);

>                  break;

>              case 2: /* Secondary context */

>                  env->dmmu.mmu_secondary_context = val;

>                  /* can be optimized to only flush MMU_USER_SECONDARY_IDX

>                     and MMU_KERNEL_SECONDARY_IDX entries */

> -                tlb_flush(CPU(cpu));

> +                tlb_flush(cs);

>                  break;

>              case 5: /* TSB access */

>                  DPRINTF_MMU("dmmu TSB write: 0x%016" PRIx64 " -> 0x%016"

> @@ -1768,13 +1765,13 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val,

>            case 1:

>                env->dmmu.mmu_primary_context = val;

>                env->immu.mmu_primary_context = val;

> -              tlb_flush_by_mmuidx(CPU(cpu),

> +              tlb_flush_by_mmuidx(cs,

>                                    (1 << MMU_USER_IDX) | (1 << MMU_KERNEL_IDX));

>                break;

>            case 2:

>                env->dmmu.mmu_secondary_context = val;

>                env->immu.mmu_secondary_context = val;

> -              tlb_flush_by_mmuidx(CPU(cpu),

> +              tlb_flush_by_mmuidx(cs,

>                                    (1 << MMU_USER_SECONDARY_IDX) |

>                                    (1 << MMU_KERNEL_SECONDARY_IDX));

>                break;

> diff --git a/target/sparc/mmu_helper.c b/target/sparc/mmu_helper.c

> index afcc5b617d..2cfcb6a976 100644

> --- a/target/sparc/mmu_helper.c

> +++ b/target/sparc/mmu_helper.c

> @@ -96,7 +96,7 @@ static int get_physical_address(CPUSPARCState *env, hwaddr *physical,

>      uint32_t pde;

>      int error_code = 0, is_dirty, is_user;

>      unsigned long page_offset;

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>

>      is_user = mmu_idx == MMU_USER_IDX;

>

> @@ -256,7 +256,7 @@ int sparc_cpu_handle_mmu_fault(CPUState *cs, vaddr address, int size, int rw,

>

>  target_ulong mmu_probe(CPUSPARCState *env, target_ulong address, int mmulev)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>      hwaddr pde_ptr;

>      uint32_t pde;

>

> @@ -323,7 +323,7 @@ target_ulong mmu_probe(CPUSPARCState *env, target_ulong address, int mmulev)

>

>  void dump_mmu(CPUSPARCState *env)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>      target_ulong va, va1, va2;

>      unsigned int n, m, o;

>      hwaddr pde_ptr, pa;

> @@ -482,7 +482,7 @@ static int get_physical_address_data(CPUSPARCState *env,

>                                       hwaddr *physical, int *prot,

>                                       target_ulong address, int rw, int mmu_idx)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>      unsigned int i;

>      uint64_t context;

>      uint64_t sfsr = 0;

> @@ -600,7 +600,7 @@ static int get_physical_address_code(CPUSPARCState *env,

>                                       hwaddr *physical, int *prot,

>                                       target_ulong address, int mmu_idx)

>  {

> -    CPUState *cs = CPU(sparc_env_get_cpu(env));

> +    CPUState *cs = env_cpu(env);

>      unsigned int i;

>      uint64_t context;

>      bool is_user = false;

> --

> 2.17.1

>

>
diff mbox series

Patch

diff --git a/target/sparc/cpu.h b/target/sparc/cpu.h
index 5a10680d8f..1442232cef 100644
--- a/target/sparc/cpu.h
+++ b/target/sparc/cpu.h
@@ -532,11 +532,6 @@  struct SPARCCPU {
     CPUSPARCState env;
 };
 
-static inline SPARCCPU *sparc_env_get_cpu(CPUSPARCState *env)
-{
-    return container_of(env, SPARCCPU, env);
-}
-
 #define ENV_OFFSET offsetof(SPARCCPU, env)
 
 #ifndef CONFIG_USER_ONLY
diff --git a/bsd-user/main.c b/bsd-user/main.c
index bfdcae4269..f6f76f1a64 100644
--- a/bsd-user/main.c
+++ b/bsd-user/main.c
@@ -486,7 +486,7 @@  static void flush_windows(CPUSPARCState *env)
 
 void cpu_loop(CPUSPARCState *env)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
     int trapnr, ret, syscall_nr;
     //target_siginfo_t info;
 
diff --git a/hw/sparc/leon3.c b/hw/sparc/leon3.c
index 774639af33..ef74bc81c2 100644
--- a/hw/sparc/leon3.c
+++ b/hw/sparc/leon3.c
@@ -91,7 +91,7 @@  static void leon3_set_pil_in(void *opaque, uint32_t pil_in)
 
                 env->interrupt_index = TT_EXTINT | i;
                 if (old_interrupt != env->interrupt_index) {
-                    cs = CPU(sparc_env_get_cpu(env));
+                    cs = env_cpu(env);
                     trace_leon3_set_irq(i);
                     cpu_interrupt(cs, CPU_INTERRUPT_HARD);
                 }
@@ -99,7 +99,7 @@  static void leon3_set_pil_in(void *opaque, uint32_t pil_in)
             }
         }
     } else if (!env->pil_in && (env->interrupt_index & ~15) == TT_EXTINT) {
-        cs = CPU(sparc_env_get_cpu(env));
+        cs = env_cpu(env);
         trace_leon3_reset_irq(env->interrupt_index & 15);
         env->interrupt_index = 0;
         cpu_reset_interrupt(cs, CPU_INTERRUPT_HARD);
diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c
index ca1e3825d5..a87bef6d4f 100644
--- a/hw/sparc/sun4m.c
+++ b/hw/sparc/sun4m.c
@@ -147,7 +147,7 @@  void cpu_check_irqs(CPUSPARCState *env)
 
                 env->interrupt_index = TT_EXTINT | i;
                 if (old_interrupt != env->interrupt_index) {
-                    cs = CPU(sparc_env_get_cpu(env));
+                    cs = env_cpu(env);
                     trace_sun4m_cpu_interrupt(i);
                     cpu_interrupt(cs, CPU_INTERRUPT_HARD);
                 }
@@ -155,7 +155,7 @@  void cpu_check_irqs(CPUSPARCState *env)
             }
         }
     } else if (!env->pil_in && (env->interrupt_index & ~15) == TT_EXTINT) {
-        cs = CPU(sparc_env_get_cpu(env));
+        cs = env_cpu(env);
         trace_sun4m_cpu_reset_interrupt(env->interrupt_index & 15);
         env->interrupt_index = 0;
         cpu_reset_interrupt(cs, CPU_INTERRUPT_HARD);
diff --git a/hw/sparc64/sparc64.c b/hw/sparc64/sparc64.c
index 408388945e..689801f37d 100644
--- a/hw/sparc64/sparc64.c
+++ b/hw/sparc64/sparc64.c
@@ -46,7 +46,7 @@  void cpu_check_irqs(CPUSPARCState *env)
     if (env->ivec_status & 0x20) {
         return;
     }
-    cs = CPU(sparc_env_get_cpu(env));
+    cs = env_cpu(env);
     /* check if TM or SM in SOFTINT are set
        setting these also causes interrupt 14 */
     if (env->softint & (SOFTINT_TIMER | SOFTINT_STIMER)) {
diff --git a/linux-user/sparc/cpu_loop.c b/linux-user/sparc/cpu_loop.c
index 9e357229c0..d85359037c 100644
--- a/linux-user/sparc/cpu_loop.c
+++ b/linux-user/sparc/cpu_loop.c
@@ -145,7 +145,7 @@  static void flush_windows(CPUSPARCState *env)
 
 void cpu_loop (CPUSPARCState *env)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
     int trapnr;
     abi_long ret;
     target_siginfo_t info;
diff --git a/target/sparc/fop_helper.c b/target/sparc/fop_helper.c
index b6642fd1d7..9eb9b75718 100644
--- a/target/sparc/fop_helper.c
+++ b/target/sparc/fop_helper.c
@@ -53,7 +53,7 @@  static target_ulong do_check_ieee_exceptions(CPUSPARCState *env, uintptr_t ra)
         }
 
         if ((fsr & FSR_CEXC_MASK) & ((fsr & FSR_TEM_MASK) >> 23)) {
-            CPUState *cs = CPU(sparc_env_get_cpu(env));
+            CPUState *cs = env_cpu(env);
 
             /* Unmasked exception, generate a trap.  Note that while
                the helper is marked as NO_WG, we can get away with
diff --git a/target/sparc/helper.c b/target/sparc/helper.c
index 46232788c8..1a52061fbf 100644
--- a/target/sparc/helper.c
+++ b/target/sparc/helper.c
@@ -26,7 +26,7 @@ 
 
 void cpu_raise_exception_ra(CPUSPARCState *env, int tt, uintptr_t ra)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
 
     cs->exception_index = tt;
     cpu_loop_exit_restore(cs, ra);
@@ -34,7 +34,7 @@  void cpu_raise_exception_ra(CPUSPARCState *env, int tt, uintptr_t ra)
 
 void helper_raise_exception(CPUSPARCState *env, int tt)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
 
     cs->exception_index = tt;
     cpu_loop_exit(cs);
@@ -42,7 +42,7 @@  void helper_raise_exception(CPUSPARCState *env, int tt)
 
 void helper_debug(CPUSPARCState *env)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
 
     cs->exception_index = EXCP_DEBUG;
     cpu_loop_exit(cs);
@@ -243,7 +243,7 @@  target_ulong helper_tsubcctv(CPUSPARCState *env, target_ulong src1,
 #ifndef TARGET_SPARC64
 void helper_power_down(CPUSPARCState *env)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
 
     cs->halted = 1;
     cs->exception_index = EXCP_HLT;
diff --git a/target/sparc/ldst_helper.c b/target/sparc/ldst_helper.c
index a7fcb84ac0..a2d033a61f 100644
--- a/target/sparc/ldst_helper.c
+++ b/target/sparc/ldst_helper.c
@@ -122,13 +122,13 @@  static uint64_t ultrasparc_tag_target(uint64_t tag_access_register)
 
 static void replace_tlb_entry(SparcTLBEntry *tlb,
                               uint64_t tlb_tag, uint64_t tlb_tte,
-                              CPUSPARCState *env1)
+                              CPUSPARCState *env)
 {
     target_ulong mask, size, va, offset;
 
     /* flush page range if translation is valid */
     if (TTE_IS_VALID(tlb->tte)) {
-        CPUState *cs = CPU(sparc_env_get_cpu(env1));
+        CPUState *cs = env_cpu(env);
 
         size = 8192ULL << 3 * TTE_PGSIZE(tlb->tte);
         mask = 1ULL + ~size;
@@ -499,7 +499,7 @@  uint64_t helper_ld_asi(CPUSPARCState *env, target_ulong addr,
 {
     int size = 1 << (memop & MO_SIZE);
     int sign = memop & MO_SIGN;
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
     uint64_t ret = 0;
 #if defined(DEBUG_MXCC) || defined(DEBUG_ASI)
     uint32_t last_addr = addr;
@@ -725,8 +725,7 @@  void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,
                    int asi, uint32_t memop)
 {
     int size = 1 << (memop & MO_SIZE);
-    SPARCCPU *cpu = sparc_env_get_cpu(env);
-    CPUState *cs = CPU(cpu);
+    CPUState *cs = env_cpu(env);
 
     do_check_align(env, addr, size - 1, GETPC());
     switch (asi) {
@@ -874,13 +873,13 @@  void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,
             DPRINTF_MMU("mmu flush level %d\n", mmulev);
             switch (mmulev) {
             case 0: /* flush page */
-                tlb_flush_page(CPU(cpu), addr & 0xfffff000);
+                tlb_flush_page(cs, addr & 0xfffff000);
                 break;
             case 1: /* flush segment (256k) */
             case 2: /* flush region (16M) */
             case 3: /* flush context (4G) */
             case 4: /* flush entire */
-                tlb_flush(CPU(cpu));
+                tlb_flush(cs);
                 break;
             default:
                 break;
@@ -905,7 +904,7 @@  void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,
                    are invalid in normal mode.  */
                 if ((oldreg ^ env->mmuregs[reg])
                     & (MMU_NF | env->def.mmu_bm)) {
-                    tlb_flush(CPU(cpu));
+                    tlb_flush(cs);
                 }
                 break;
             case 1: /* Context Table Pointer Register */
@@ -916,7 +915,7 @@  void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,
                 if (oldreg != env->mmuregs[reg]) {
                     /* we flush when the MMU context changes because
                        QEMU has no MMU context support */
-                    tlb_flush(CPU(cpu));
+                    tlb_flush(cs);
                 }
                 break;
             case 3: /* Synchronous Fault Status Register with Clear */
@@ -1027,8 +1026,7 @@  void helper_st_asi(CPUSPARCState *env, target_ulong addr, uint64_t val,
     case ASI_USERTXT: /* User code access, XXX */
     case ASI_KERNELTXT: /* Supervisor code access, XXX */
     default:
-        cpu_unassigned_access(CPU(sparc_env_get_cpu(env)),
-                              addr, true, false, asi, size);
+        cpu_unassigned_access(cs, addr, true, false, asi, size);
         break;
 
     case ASI_USERDATA: /* User data access */
@@ -1175,7 +1173,7 @@  uint64_t helper_ld_asi(CPUSPARCState *env, target_ulong addr,
 {
     int size = 1 << (memop & MO_SIZE);
     int sign = memop & MO_SIGN;
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
     uint64_t ret = 0;
 #if defined(DEBUG_ASI)
     target_ulong last_addr = addr;
@@ -1481,8 +1479,7 @@  void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val,
                    int asi, uint32_t memop)
 {
     int size = 1 << (memop & MO_SIZE);
-    SPARCCPU *cpu = sparc_env_get_cpu(env);
-    CPUState *cs = CPU(cpu);
+    CPUState *cs = env_cpu(env);
 
 #ifdef DEBUG_ASI
     dump_asi("write", addr, asi, size, val);
@@ -1686,13 +1683,13 @@  void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val,
                 env->dmmu.mmu_primary_context = val;
                 /* can be optimized to only flush MMU_USER_IDX
                    and MMU_KERNEL_IDX entries */
-                tlb_flush(CPU(cpu));
+                tlb_flush(cs);
                 break;
             case 2: /* Secondary context */
                 env->dmmu.mmu_secondary_context = val;
                 /* can be optimized to only flush MMU_USER_SECONDARY_IDX
                    and MMU_KERNEL_SECONDARY_IDX entries */
-                tlb_flush(CPU(cpu));
+                tlb_flush(cs);
                 break;
             case 5: /* TSB access */
                 DPRINTF_MMU("dmmu TSB write: 0x%016" PRIx64 " -> 0x%016"
@@ -1768,13 +1765,13 @@  void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val,
           case 1:
               env->dmmu.mmu_primary_context = val;
               env->immu.mmu_primary_context = val;
-              tlb_flush_by_mmuidx(CPU(cpu),
+              tlb_flush_by_mmuidx(cs,
                                   (1 << MMU_USER_IDX) | (1 << MMU_KERNEL_IDX));
               break;
           case 2:
               env->dmmu.mmu_secondary_context = val;
               env->immu.mmu_secondary_context = val;
-              tlb_flush_by_mmuidx(CPU(cpu),
+              tlb_flush_by_mmuidx(cs,
                                   (1 << MMU_USER_SECONDARY_IDX) |
                                   (1 << MMU_KERNEL_SECONDARY_IDX));
               break;
diff --git a/target/sparc/mmu_helper.c b/target/sparc/mmu_helper.c
index afcc5b617d..2cfcb6a976 100644
--- a/target/sparc/mmu_helper.c
+++ b/target/sparc/mmu_helper.c
@@ -96,7 +96,7 @@  static int get_physical_address(CPUSPARCState *env, hwaddr *physical,
     uint32_t pde;
     int error_code = 0, is_dirty, is_user;
     unsigned long page_offset;
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
 
     is_user = mmu_idx == MMU_USER_IDX;
 
@@ -256,7 +256,7 @@  int sparc_cpu_handle_mmu_fault(CPUState *cs, vaddr address, int size, int rw,
 
 target_ulong mmu_probe(CPUSPARCState *env, target_ulong address, int mmulev)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
     hwaddr pde_ptr;
     uint32_t pde;
 
@@ -323,7 +323,7 @@  target_ulong mmu_probe(CPUSPARCState *env, target_ulong address, int mmulev)
 
 void dump_mmu(CPUSPARCState *env)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
     target_ulong va, va1, va2;
     unsigned int n, m, o;
     hwaddr pde_ptr, pa;
@@ -482,7 +482,7 @@  static int get_physical_address_data(CPUSPARCState *env,
                                      hwaddr *physical, int *prot,
                                      target_ulong address, int rw, int mmu_idx)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
     unsigned int i;
     uint64_t context;
     uint64_t sfsr = 0;
@@ -600,7 +600,7 @@  static int get_physical_address_code(CPUSPARCState *env,
                                      hwaddr *physical, int *prot,
                                      target_ulong address, int mmu_idx)
 {
-    CPUState *cs = CPU(sparc_env_get_cpu(env));
+    CPUState *cs = env_cpu(env);
     unsigned int i;
     uint64_t context;
     bool is_user = false;