On Mon May 27, 2024 at 9:13 AM AEST, BALATON Zoltan wrote: > There is already a hash32_bat_prot() function that does most if this > and the rest can be inlined. Export hash32_bat_prot() and rename it to > ppc_hash32_bat_prot() to match other functions and use it in > get_bat_6xx_tlb().
Reviewed-by: Nicholas Piggin <npig...@gmail.com> > > Signed-off-by: BALATON Zoltan <bala...@eik.bme.hu> > --- > target/ppc/mmu-hash32.c | 18 +------------- > target/ppc/mmu-hash32.h | 14 +++++++++++ > target/ppc/mmu_common.c | 52 ++++++++++------------------------------- > 3 files changed, 27 insertions(+), 57 deletions(-) > > diff --git a/target/ppc/mmu-hash32.c b/target/ppc/mmu-hash32.c > index 160311de87..6f0f0bbb00 100644 > --- a/target/ppc/mmu-hash32.c > +++ b/target/ppc/mmu-hash32.c > @@ -48,22 +48,6 @@ static target_ulong hash32_bat_size(int mmu_idx, > return BATU32_BEPI & ~((batu & BATU32_BL) << 15); > } > > -static int hash32_bat_prot(PowerPCCPU *cpu, > - target_ulong batu, target_ulong batl) > -{ > - int pp, prot; > - > - prot = 0; > - pp = batl & BATL32_PP; > - if (pp != 0) { > - prot = PAGE_READ | PAGE_EXEC; > - if (pp == 0x2) { > - prot |= PAGE_WRITE; > - } > - } > - return prot; > -} > - > static hwaddr ppc_hash32_bat_lookup(PowerPCCPU *cpu, target_ulong ea, > MMUAccessType access_type, int *prot, > int mmu_idx) > @@ -95,7 +79,7 @@ static hwaddr ppc_hash32_bat_lookup(PowerPCCPU *cpu, > target_ulong ea, > if (mask && ((ea & mask) == (batu & BATU32_BEPI))) { > hwaddr raddr = (batl & mask) | (ea & ~mask); > > - *prot = hash32_bat_prot(cpu, batu, batl); > + *prot = ppc_hash32_bat_prot(batu, batl); > > return raddr & TARGET_PAGE_MASK; > } > diff --git a/target/ppc/mmu-hash32.h b/target/ppc/mmu-hash32.h > index 5902cf8333..bd75f7d647 100644 > --- a/target/ppc/mmu-hash32.h > +++ b/target/ppc/mmu-hash32.h > @@ -143,6 +143,20 @@ static inline int ppc_hash32_prot(bool key, int pp, bool > nx) > return nx ? prot : prot | PAGE_EXEC; > } > > +static inline int ppc_hash32_bat_prot(target_ulong batu, target_ulong batl) > +{ > + int prot = 0; > + int pp = batl & BATL32_PP; > + > + if (pp) { > + prot = PAGE_READ | PAGE_EXEC; > + if (pp == 0x2) { > + prot |= PAGE_WRITE; > + } > + } > + return prot; > +} > + > typedef struct { > uint32_t pte0, pte1; > } ppc_hash_pte32_t; > diff --git a/target/ppc/mmu_common.c b/target/ppc/mmu_common.c > index aa002bba35..624ed51a92 100644 > --- a/target/ppc/mmu_common.c > +++ b/target/ppc/mmu_common.c > @@ -193,40 +193,13 @@ static int ppc6xx_tlb_check(CPUPPCState *env, hwaddr > *raddr, int *prot, > return ret; > } > > -/* Perform BAT hit & translation */ > -static inline void bat_size_prot(CPUPPCState *env, target_ulong *blp, > - int *validp, int *protp, target_ulong *BATu, > - target_ulong *BATl) > -{ > - target_ulong bl; > - int pp, valid, prot; > - > - bl = (*BATu & BATU32_BL) << 15; > - valid = 0; > - prot = 0; > - if ((!FIELD_EX64(env->msr, MSR, PR) && (*BATu & 0x00000002)) || > - (FIELD_EX64(env->msr, MSR, PR) && (*BATu & 0x00000001))) { > - valid = 1; > - pp = *BATl & 0x00000003; > - if (pp != 0) { > - prot = PAGE_READ | PAGE_EXEC; > - if (pp == 0x2) { > - prot |= PAGE_WRITE; > - } > - } > - } > - *blp = bl; > - *validp = valid; > - *protp = prot; > -} > - > static int get_bat_6xx_tlb(CPUPPCState *env, mmu_ctx_t *ctx, > - target_ulong eaddr, MMUAccessType access_type) > + target_ulong eaddr, MMUAccessType access_type, > + bool pr) > { > target_ulong *BATlt, *BATut, *BATu, *BATl; > target_ulong BEPIl, BEPIu, bl; > - int i, valid, prot; > - int ret = -1; > + int i, ret = -1; > bool ifetch = access_type == MMU_INST_FETCH; > > qemu_log_mask(CPU_LOG_MMU, "%s: %cBAT v " TARGET_FMT_lx "\n", __func__, > @@ -243,20 +216,19 @@ static int get_bat_6xx_tlb(CPUPPCState *env, mmu_ctx_t > *ctx, > BATl = &BATlt[i]; > BEPIu = *BATu & BATU32_BEPIU; > BEPIl = *BATu & BATU32_BEPIL; > - bat_size_prot(env, &bl, &valid, &prot, BATu, BATl); > qemu_log_mask(CPU_LOG_MMU, "%s: %cBAT%d v " TARGET_FMT_lx " BATu " > TARGET_FMT_lx " BATl " TARGET_FMT_lx "\n", __func__, > ifetch ? 'I' : 'D', i, eaddr, *BATu, *BATl); > - if ((eaddr & BATU32_BEPIU) == BEPIu && > - ((eaddr & BATU32_BEPIL) & ~bl) == BEPIl) { > - /* BAT matches */ > - if (valid != 0) { > + bl = (*BATu & BATU32_BL) << 15; > + if ((!pr && (*BATu & BATU32_VS)) || (pr && (*BATu & BATU32_VP))) { > + if ((eaddr & BATU32_BEPIU) == BEPIu && > + ((eaddr & BATU32_BEPIL) & ~bl) == BEPIl) { > /* Get physical address */ > ctx->raddr = (*BATl & BATU32_BEPIU) | > ((eaddr & BATU32_BEPIL & bl) | (*BATl & BATU32_BEPIL)) | > (eaddr & 0x0001F000); > /* Compute access rights */ > - ctx->prot = prot; > + ctx->prot = ppc_hash32_bat_prot(*BATu, *BATl); > if (check_prot_access_type(ctx->prot, access_type)) { > qemu_log_mask(CPU_LOG_MMU, "BAT %d match: r " > HWADDR_FMT_plx > " prot=%c%c\n", i, ctx->raddr, > @@ -300,16 +272,16 @@ static int mmu6xx_get_physical_address(CPUPPCState > *env, mmu_ctx_t *ctx, > PowerPCCPU *cpu = env_archcpu(env); > hwaddr hash; > target_ulong vsid, sr, pgidx, ptem; > - bool key, pr, ds, nx; > + bool key, ds, nx; > + bool pr = FIELD_EX64(env->msr, MSR, PR); > > /* First try to find a BAT entry if there are any */ > - if (env->nb_BATs && get_bat_6xx_tlb(env, ctx, eaddr, access_type) == 0) { > + if (env->nb_BATs && > + get_bat_6xx_tlb(env, ctx, eaddr, access_type, pr) == 0) { > return 0; > } > > /* Perform segment based translation when no BATs matched */ > - pr = FIELD_EX64(env->msr, MSR, PR); > - > sr = env->sr[eaddr >> 28]; > key = ppc_hash32_key(pr, sr); > *keyp = key;