From nobody Wed Oct 1 22:33:20 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 357B522A4F1; Sun, 28 Sep 2025 15:02:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759071776; cv=none; b=PmT7x7iVH3Lxzig26HdB+QlCth1NW+u0KXE9KIy5t28anN3hNdTrIPqK9GJe9ZKwUSzWvvOnfDBBi2M1fU4Y9Y9qrXuNHPD7RJTuLSVRIkkYe9IeqPoLfDcckLn+eikYBvuNIkwF6X2qJzPzBANwcmcuIMTtkYUCjGuK3h07RnE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759071776; c=relaxed/simple; bh=jNtnwWwMV3plKAecUNIdrhRBaM4ED7uOfKosdK+4834=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=TVbs5MMzbsNVaFPs3tVVSVw83IWK9khMC3LXPd6kI67XST2byb85JQ/nhhStrZQGUAYcq5MveBcLQ/mY6Ahb33MJXAiLfvefBenn9pZgmDPj2ThfnAc+hlktQXKyG6TQBOVDFXl0HF2Dj1T6aZsKcySjVdE98AuKML6yda7M8IU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=q5iYL8J4; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="q5iYL8J4" Received: by smtp.kernel.org (Postfix) with ESMTPSA id E2C49C4CEF5; Sun, 28 Sep 2025 15:02:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1759071775; bh=jNtnwWwMV3plKAecUNIdrhRBaM4ED7uOfKosdK+4834=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=q5iYL8J4WbMGQv508j0QZDEfRJ0+7iu0lvJK3NuWtN1V3QOH6cmyo6Kk/sloMCjGJ 3opzab96iV6QBOWtv3sEcwe3ixvE9omNE1shesZU9PSudr5AtxFOSo+XfXeHrS3kRm N4PgnJV5qrM/PtIMgPQ1CS8vRmu6FNYaMEtf1Uln3Tgfwxf4/ZiydAtEBi+XdEIyaT tFEnF9SvQ2T9pHurTpXc/xL+PlNqDJ6+SkERg2O08QiR0KVTHkZeHs+oryBHUZdFXG lNYCWoQiMxLiaXDpAfuLbBo4vqWn76vCduW5o0tt6jpMIf/+MzpS5eMBgsF2hYeqvS WZi3vUeeyRjAw== From: Leon Romanovsky To: Marek Szyprowski Cc: Leon Romanovsky , Jason Gunthorpe , Andreas Larsson , Borislav Petkov , Dave Hansen , "David S. Miller" , Geoff Levand , Helge Deller , Ingo Molnar , iommu@lists.linux.dev, "James E.J. Bottomley" , Jason Wang , Juergen Gross , linux-alpha@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mips@vger.kernel.org, linux-parisc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Madhavan Srinivasan , Matt Turner , Michael Ellerman , "Michael S. Tsirkin" , Richard Henderson , sparclinux@vger.kernel.org, Stefano Stabellini , Thomas Bogendoerfer , Thomas Gleixner , virtualization@lists.linux.dev, x86@kernel.org, xen-devel@lists.xenproject.org, Magnus Lindholm Subject: [PATCH v1 1/9] alpha: Convert mapping routine to rely on physical address Date: Sun, 28 Sep 2025 18:02:21 +0300 Message-ID: <512d4c498103fcfccd8c60ce1982cd961434d30b.1759071169.git.leon@kernel.org> X-Mailer: git-send-email 2.51.0 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Leon Romanovsky Alpha doesn't need struct *page and can perform mapping based on physical addresses. So convert it to implement new .map_phys callback. As part of this change, remove useless BUG_ON() as DMA mapping layer ensures that right direction is provided. Signed-off-by: Leon Romanovsky Tested-by: Magnus Lindholm --- arch/alpha/kernel/pci_iommu.c | 48 +++++++++++++++-------------------- 1 file changed, 21 insertions(+), 27 deletions(-) diff --git a/arch/alpha/kernel/pci_iommu.c b/arch/alpha/kernel/pci_iommu.c index dc91de50f906..3e4f631a1f27 100644 --- a/arch/alpha/kernel/pci_iommu.c +++ b/arch/alpha/kernel/pci_iommu.c @@ -224,28 +224,26 @@ static int pci_dac_dma_supported(struct pci_dev *dev,= u64 mask) until either pci_unmap_single or pci_dma_sync_single is performed. */ =20 static dma_addr_t -pci_map_single_1(struct pci_dev *pdev, void *cpu_addr, size_t size, +pci_map_single_1(struct pci_dev *pdev, phys_addr_t paddr, size_t size, int dac_allowed) { struct pci_controller *hose =3D pdev ? pdev->sysdata : pci_isa_hose; dma_addr_t max_dma =3D pdev ? pdev->dma_mask : ISA_DMA_MASK; + unsigned long offset =3D offset_in_page(paddr); struct pci_iommu_arena *arena; long npages, dma_ofs, i; - unsigned long paddr; dma_addr_t ret; unsigned int align =3D 0; struct device *dev =3D pdev ? &pdev->dev : NULL; =20 - paddr =3D __pa(cpu_addr); - #if !DEBUG_NODIRECT /* First check to see if we can use the direct map window. */ if (paddr + size + __direct_map_base - 1 <=3D max_dma && paddr + size <=3D __direct_map_size) { ret =3D paddr + __direct_map_base; =20 - DBGA2("pci_map_single: [%p,%zx] -> direct %llx from %ps\n", - cpu_addr, size, ret, __builtin_return_address(0)); + DBGA2("pci_map_single: [%pa,%zx] -> direct %llx from %ps\n", + &paddr, size, ret, __builtin_return_address(0)); =20 return ret; } @@ -255,8 +253,8 @@ pci_map_single_1(struct pci_dev *pdev, void *cpu_addr, = size_t size, if (dac_allowed) { ret =3D paddr + alpha_mv.pci_dac_offset; =20 - DBGA2("pci_map_single: [%p,%zx] -> DAC %llx from %ps\n", - cpu_addr, size, ret, __builtin_return_address(0)); + DBGA2("pci_map_single: [%pa,%zx] -> DAC %llx from %ps\n", + &paddr, size, ret, __builtin_return_address(0)); =20 return ret; } @@ -290,10 +288,10 @@ pci_map_single_1(struct pci_dev *pdev, void *cpu_addr= , size_t size, arena->ptes[i + dma_ofs] =3D mk_iommu_pte(paddr); =20 ret =3D arena->dma_base + dma_ofs * PAGE_SIZE; - ret +=3D (unsigned long)cpu_addr & ~PAGE_MASK; + ret +=3D offset; =20 - DBGA2("pci_map_single: [%p,%zx] np %ld -> sg %llx from %ps\n", - cpu_addr, size, npages, ret, __builtin_return_address(0)); + DBGA2("pci_map_single: [%pa,%zx] np %ld -> sg %llx from %ps\n", + &paddr, size, npages, ret, __builtin_return_address(0)); =20 return ret; } @@ -322,19 +320,18 @@ static struct pci_dev *alpha_gendev_to_pci(struct dev= ice *dev) return NULL; } =20 -static dma_addr_t alpha_pci_map_page(struct device *dev, struct page *page, - unsigned long offset, size_t size, - enum dma_data_direction dir, +static dma_addr_t alpha_pci_map_phys(struct device *dev, phys_addr_t phys, + size_t size, enum dma_data_direction dir, unsigned long attrs) { struct pci_dev *pdev =3D alpha_gendev_to_pci(dev); int dac_allowed; =20 - BUG_ON(dir =3D=3D DMA_NONE); + if (attrs & DMA_ATTR_MMIO) + return DMA_MAPPING_ERROR; =20 - dac_allowed =3D pdev ? pci_dac_dma_supported(pdev, pdev->dma_mask) : 0;=20 - return pci_map_single_1(pdev, (char *)page_address(page) + offset,=20 - size, dac_allowed); + dac_allowed =3D pdev ? pci_dac_dma_supported(pdev, pdev->dma_mask) : 0; + return pci_map_single_1(pdev, phys, size, dac_allowed); } =20 /* Unmap a single streaming mode DMA translation. The DMA_ADDR and @@ -343,7 +340,7 @@ static dma_addr_t alpha_pci_map_page(struct device *dev= , struct page *page, the cpu to the buffer are guaranteed to see whatever the device wrote there. */ =20 -static void alpha_pci_unmap_page(struct device *dev, dma_addr_t dma_addr, +static void alpha_pci_unmap_phys(struct device *dev, dma_addr_t dma_addr, size_t size, enum dma_data_direction dir, unsigned long attrs) { @@ -353,8 +350,6 @@ static void alpha_pci_unmap_page(struct device *dev, dm= a_addr_t dma_addr, struct pci_iommu_arena *arena; long dma_ofs, npages; =20 - BUG_ON(dir =3D=3D DMA_NONE); - if (dma_addr >=3D __direct_map_base && dma_addr < __direct_map_base + __direct_map_size) { /* Nothing to do. */ @@ -429,7 +424,7 @@ static void *alpha_pci_alloc_coherent(struct device *de= v, size_t size, } memset(cpu_addr, 0, size); =20 - *dma_addrp =3D pci_map_single_1(pdev, cpu_addr, size, 0); + *dma_addrp =3D pci_map_single_1(pdev, virt_to_phys(cpu_addr), size, 0); if (*dma_addrp =3D=3D DMA_MAPPING_ERROR) { free_pages((unsigned long)cpu_addr, order); if (alpha_mv.mv_pci_tbi || (gfp & GFP_DMA)) @@ -643,9 +638,8 @@ static int alpha_pci_map_sg(struct device *dev, struct = scatterlist *sg, /* Fast path single entry scatterlists. */ if (nents =3D=3D 1) { sg->dma_length =3D sg->length; - sg->dma_address - =3D pci_map_single_1(pdev, SG_ENT_VIRT_ADDRESS(sg), - sg->length, dac_allowed); + sg->dma_address =3D pci_map_single_1(pdev, sg_phys(sg), + sg->length, dac_allowed); if (sg->dma_address =3D=3D DMA_MAPPING_ERROR) return -EIO; return 1; @@ -917,8 +911,8 @@ iommu_unbind(struct pci_iommu_arena *arena, long pg_sta= rt, long pg_count) const struct dma_map_ops alpha_pci_ops =3D { .alloc =3D alpha_pci_alloc_coherent, .free =3D alpha_pci_free_coherent, - .map_page =3D alpha_pci_map_page, - .unmap_page =3D alpha_pci_unmap_page, + .map_phys =3D alpha_pci_map_phys, + .unmap_phys =3D alpha_pci_unmap_phys, .map_sg =3D alpha_pci_map_sg, .unmap_sg =3D alpha_pci_unmap_sg, .dma_supported =3D alpha_pci_supported, --=20 2.51.0