target/ppc/mmu-radix64.c: Drop a local variable
The value is only used once so no need to introduce a local variable for it. Reviewed-by: Nicholas Piggin <npiggin@gmail.com> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
This commit is contained in:
parent
e89b0629b9
commit
950251ee7b
@ -185,7 +185,6 @@ static bool ppc_radix64_check_prot(PowerPCCPU *cpu, MMUAccessType access_type,
|
|||||||
int mmu_idx, bool partition_scoped)
|
int mmu_idx, bool partition_scoped)
|
||||||
{
|
{
|
||||||
CPUPPCState *env = &cpu->env;
|
CPUPPCState *env = &cpu->env;
|
||||||
int need_prot;
|
|
||||||
|
|
||||||
/* Check Page Attributes (pte58:59) */
|
/* Check Page Attributes (pte58:59) */
|
||||||
if ((pte & R_PTE_ATT) == R_PTE_ATT_NI_IO && access_type == MMU_INST_FETCH) {
|
if ((pte & R_PTE_ATT) == R_PTE_ATT_NI_IO && access_type == MMU_INST_FETCH) {
|
||||||
@ -210,8 +209,8 @@ static bool ppc_radix64_check_prot(PowerPCCPU *cpu, MMUAccessType access_type,
|
|||||||
}
|
}
|
||||||
|
|
||||||
/* Check if requested access type is allowed */
|
/* Check if requested access type is allowed */
|
||||||
need_prot = prot_for_access_type(access_type);
|
if (prot_for_access_type(access_type) & ~*prot) {
|
||||||
if (need_prot & ~*prot) { /* Page Protected for that Access */
|
/* Page Protected for that Access */
|
||||||
*fault_cause |= access_type == MMU_INST_FETCH ? SRR1_NOEXEC_GUARD :
|
*fault_cause |= access_type == MMU_INST_FETCH ? SRR1_NOEXEC_GUARD :
|
||||||
DSISR_PROTFAULT;
|
DSISR_PROTFAULT;
|
||||||
return true;
|
return true;
|
||||||
|
Loading…
Reference in New Issue
Block a user