diff --git a/xen/arch/riscv/include/asm/mm.h b/xen/arch/riscv/include/asm/mm.h index 9e28c24954..1ac66283ec 100644 --- a/xen/arch/riscv/include/asm/mm.h +++ b/xen/arch/riscv/include/asm/mm.h @@ -22,9 +22,10 @@ extern vaddr_t directmap_virt_start; #define paddr_to_pfn(pa) ((unsigned long)((pa) >> PAGE_SHIFT)) static inline pte_t paddr_to_pte(paddr_t paddr, - unsigned int permissions) + unsigned int pte_flags) { - return (pte_t) { .pte = (paddr_to_pfn(paddr) << PTE_PPN_SHIFT) | permissions }; + return (pte_t) { .pte = (paddr_to_pfn(paddr) << PTE_PPN_SHIFT) | + pte_flags }; } static inline paddr_t pte_to_paddr(pte_t pte) diff --git a/xen/arch/riscv/mm.c b/xen/arch/riscv/mm.c index 610d111945..06f0406781 100644 --- a/xen/arch/riscv/mm.c +++ b/xen/arch/riscv/mm.c @@ -140,7 +140,7 @@ static void __init setup_initial_mapping(struct mmu_desc *mmu_desc, case 1: /* Level 0 */ { unsigned long paddr = (page_addr - map_start) + pa_start; - unsigned int permissions = PTE_LEAF_DEFAULT; + unsigned int pte_flags = PTE_LEAF_DEFAULT; unsigned long addr = is_identity_mapping ? page_addr : virt_to_maddr(page_addr); pte_t pte_to_be_written; @@ -149,13 +149,12 @@ static void __init setup_initial_mapping(struct mmu_desc *mmu_desc, if ( is_kernel_text(addr) || is_kernel_inittext(addr) ) - permissions = - PTE_EXECUTABLE | PTE_READABLE | PTE_VALID; + pte_flags = PTE_EXECUTABLE | PTE_READABLE | PTE_VALID; if ( is_kernel_rodata(addr) ) - permissions = PTE_READABLE | PTE_VALID; + pte_flags = PTE_READABLE | PTE_VALID; - pte_to_be_written = paddr_to_pte(paddr, permissions); + pte_to_be_written = paddr_to_pte(paddr, pte_flags); if ( !pte_is_valid(pgtbl[index]) ) pgtbl[index] = pte_to_be_written; diff --git a/xen/arch/riscv/p2m.c b/xen/arch/riscv/p2m.c index 1cea86512c..f7b380b90a 100644 --- a/xen/arch/riscv/p2m.c +++ b/xen/arch/riscv/p2m.c @@ -584,7 +584,7 @@ static inline void p2m_clean_pte(pte_t *p, bool clean_cache) p2m_write_pte(p, pte, clean_cache); } -static void p2m_set_permission(pte_t *e, p2m_type_t t) +static void p2m_set_pte_flags(pte_t *e, p2m_type_t t) { e->pte &= ~PTE_ACCESS_MASK; @@ -676,7 +676,7 @@ static pte_t p2m_pte_from_mfn(mfn_t mfn, p2m_type_t t, break; } - p2m_set_permission(&e, t); + p2m_set_pte_flags(&e, t); p2m_set_type(&e, t, ctx); } else