On Mon May 27, 2024 at 9:13 AM AEST, BALATON Zoltan wrote: > This function takes PowerPCCPU but only needs the env from it. Change > its parameter to CPUPPCState *env.
Acked-by: Nicholas Piggin <npig...@gmail.com> > > Signed-off-by: BALATON Zoltan <bala...@eik.bme.hu> > --- > hw/ppc/spapr_rtas.c | 2 +- > target/ppc/mmu-book3s-v3.h | 4 ++-- > target/ppc/mmu_common.c | 4 ++-- > 3 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/hw/ppc/spapr_rtas.c b/hw/ppc/spapr_rtas.c > index f329693c55..38e94fc0d7 100644 > --- a/hw/ppc/spapr_rtas.c > +++ b/hw/ppc/spapr_rtas.c > @@ -177,7 +177,7 @@ static void rtas_start_cpu(PowerPCCPU *callcpu, > SpaprMachineState *spapr, > * New cpus are expected to start in the same radix/hash mode > * as the existing CPUs > */ > - if (ppc64_v3_radix(callcpu)) { > + if (ppc64_v3_radix(&callcpu->env)) { > lpcr |= LPCR_UPRT | LPCR_GTSE | LPCR_HR; > } else { > lpcr &= ~(LPCR_UPRT | LPCR_GTSE | LPCR_HR); > diff --git a/target/ppc/mmu-book3s-v3.h b/target/ppc/mmu-book3s-v3.h > index be66e26604..e52129ff7f 100644 > --- a/target/ppc/mmu-book3s-v3.h > +++ b/target/ppc/mmu-book3s-v3.h > @@ -75,9 +75,9 @@ bool ppc64_v3_get_pate(PowerPCCPU *cpu, target_ulong lpid, > * dig out the partition table in the fast path. This is > * also how the HW uses it. > */ > -static inline bool ppc64_v3_radix(PowerPCCPU *cpu) > +static inline bool ppc64_v3_radix(CPUPPCState *env) > { > - return !!(cpu->env.spr[SPR_LPCR] & LPCR_HR); > + return !!(env->spr[SPR_LPCR] & LPCR_HR); > } > > #endif /* TARGET_PPC64 */ > diff --git a/target/ppc/mmu_common.c b/target/ppc/mmu_common.c > index b45eb64f6e..ab055ca96b 100644 > --- a/target/ppc/mmu_common.c > +++ b/target/ppc/mmu_common.c > @@ -565,7 +565,7 @@ void dump_mmu(CPUPPCState *env) > dump_slb(env_archcpu(env)); > break; > case POWERPC_MMU_3_00: > - if (ppc64_v3_radix(env_archcpu(env))) { > + if (ppc64_v3_radix(env)) { > qemu_log_mask(LOG_UNIMP, "%s: the PPC64 MMU is unsupported\n", > __func__); > } else { > @@ -810,7 +810,7 @@ bool ppc_xlate(PowerPCCPU *cpu, vaddr eaddr, > MMUAccessType access_type, > switch (cpu->env.mmu_model) { > #if defined(TARGET_PPC64) > case POWERPC_MMU_3_00: > - if (ppc64_v3_radix(cpu)) { > + if (ppc64_v3_radix(&cpu->env)) { > return ppc_radix64_xlate(cpu, eaddr, access_type, raddrp, > psizep, protp, mmu_idx, guest_visible); > }