From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 74758C433F5 for ; Thu, 6 Oct 2022 11:12:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231517AbiJFLL7 (ORCPT ); Thu, 6 Oct 2022 07:11:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44008 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231496AbiJFLL4 (ORCPT ); Thu, 6 Oct 2022 07:11:56 -0400 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id EA44A8E9A2 for ; Thu, 6 Oct 2022 04:11:54 -0700 (PDT) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id D8CB11C00; Thu, 6 Oct 2022 04:12:00 -0700 (PDT) Received: from monolith.localdoman (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A25153F73B; Thu, 6 Oct 2022 04:11:53 -0700 (PDT) From: Alexandru Elisei To: pbonzini@redhat.com, thuth@redhat.com, andrew.jones@linux.dev, kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu Subject: [kvm-unit-tests PATCH 3/3] arm/arm64: mmu: Rename mmu_get_pte() -> follow_pte() Date: Thu, 6 Oct 2022 12:12:41 +0100 Message-Id: <20221006111241.15083-4-alexandru.elisei@arm.com> X-Mailer: git-send-email 2.37.3 In-Reply-To: <20221006111241.15083-1-alexandru.elisei@arm.com> References: <20221006111241.15083-1-alexandru.elisei@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org The function get_pte() from mmu.c returns a pointer to the PTE associated with the requested virtual address, mapping the virtual address in the process if it's not already mapped. mmu_get_pte() returns a pointer to the PTE if and only if the virtual is mapped in pgtable, otherwise returns NULL. Rename it to follow_pte() to avoid any confusion with get_pte(). follow_pte() also matches the name of Linux kernel function with a similar purpose. Also remove the mmu_enabled() check from the function, as the purpose of the function is to get the mapping for the virtual address in the pgtable supplied as the argument, not to translate the virtual address to a physical address using the current translation; that's what virt_to_phys() does. Signed-off-by: Alexandru Elisei --- lib/arm/asm/mmu-api.h | 2 +- lib/arm/mmu.c | 9 +++------ 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/lib/arm/asm/mmu-api.h b/lib/arm/asm/mmu-api.h index 3d77cbfd8b24..6c1136d957f9 100644 --- a/lib/arm/asm/mmu-api.h +++ b/lib/arm/asm/mmu-api.h @@ -17,6 +17,6 @@ extern void mmu_set_range_sect(pgd_t *pgtable, uintptr_t virt_offset, extern void mmu_set_range_ptes(pgd_t *pgtable, uintptr_t virt_offset, phys_addr_t phys_start, phys_addr_t phys_end, pgprot_t prot); -extern pteval_t *mmu_get_pte(pgd_t *pgtable, uintptr_t vaddr); +extern pteval_t *follow_pte(pgd_t *pgtable, uintptr_t vaddr); extern void mmu_clear_user(pgd_t *pgtable, unsigned long vaddr); #endif diff --git a/lib/arm/mmu.c b/lib/arm/mmu.c index 2aaa63d538c0..ec3ce63f2316 100644 --- a/lib/arm/mmu.c +++ b/lib/arm/mmu.c @@ -117,16 +117,13 @@ pteval_t *install_page(pgd_t *pgtable, phys_addr_t phys, void *virt) * certain conditions are met (see Arm ARM D5-2669 for AArch64 and * B3-1378 for AArch32 for more details). */ -pteval_t *mmu_get_pte(pgd_t *pgtable, uintptr_t vaddr) +pteval_t *follow_pte(pgd_t *pgtable, uintptr_t vaddr) { pgd_t *pgd; pud_t *pud; pmd_t *pmd; pte_t *pte; - if (!mmu_enabled()) - return NULL; - pgd = pgd_offset(pgtable, vaddr); if (!pgd_valid(*pgd)) return NULL; @@ -153,7 +150,7 @@ phys_addr_t virt_to_pte_phys(pgd_t *pgtable, void *virt) phys_addr_t mask; pteval_t *pteval; - pteval = mmu_get_pte(pgtable, (uintptr_t)virt); + pteval = follow_pte(pgtable, (uintptr_t)virt); if (!pteval || !pte_valid(__pte(*pteval))) { install_page(pgtable, (phys_addr_t)(unsigned long)virt, virt); return (phys_addr_t)(unsigned long)virt; @@ -284,7 +281,7 @@ unsigned long __phys_to_virt(phys_addr_t addr) void mmu_clear_user(pgd_t *pgtable, unsigned long vaddr) { - pteval_t *p_pte = mmu_get_pte(pgtable, vaddr); + pteval_t *p_pte = follow_pte(pgtable, vaddr); if (p_pte) { pteval_t entry = *p_pte & ~PTE_USER; WRITE_ONCE(*p_pte, entry); -- 2.37.0