All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lu Baolu <baolu.lu@linux.intel.com>
To: Joerg Roedel <joro@8bytes.org>, Will Deacon <will@kernel.org>,
	Robin Murphy <robin.murphy@arm.com>,
	Jason Gunthorpe <jgg@ziepe.ca>, Kevin Tian <kevin.tian@intel.com>
Cc: Yi Liu <yi.l.liu@intel.com>, David Airlie <airlied@gmail.com>,
	Daniel Vetter <daniel@ffwll.ch>, Kalle Valo <kvalo@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Mathieu Poirier <mathieu.poirier@linaro.org>,
	Alex Williamson <alex.williamson@redhat.com>,
	mst@redhat.com, Jason Wang <jasowang@redhat.com>,
	Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	Mikko Perttunen <mperttunen@nvidia.com>,
	Jeff Johnson <quic_jjohnson@quicinc.com>,
	ath10k@lists.infradead.org, ath11k@lists.infradead.org,
	iommu@lists.linux.dev, dri-devel@lists.freedesktop.org,
	linux-kernel@vger.kernel.org, Lu Baolu <baolu.lu@linux.intel.com>
Subject: [PATCH v3 15/21] iommu/vt-d: Add helper to allocate paging domain
Date: Mon, 10 Jun 2024 16:55:49 +0800	[thread overview]
Message-ID: <20240610085555.88197-16-baolu.lu@linux.intel.com> (raw)
In-Reply-To: <20240610085555.88197-1-baolu.lu@linux.intel.com>

The domain_alloc_user operation is currently implemented by allocating a
paging domain using iommu_domain_alloc(). This is because it needs to fully
initialize the domain before return. Add a helper to do this to avoid using
iommu_domain_alloc().

Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
---
 drivers/iommu/intel/iommu.c | 87 +++++++++++++++++++++++++++++++++----
 1 file changed, 78 insertions(+), 9 deletions(-)

diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c
index 2e9811bf2a4e..ccde5f5972e4 100644
--- a/drivers/iommu/intel/iommu.c
+++ b/drivers/iommu/intel/iommu.c
@@ -3633,6 +3633,79 @@ static struct iommu_domain blocking_domain = {
 	}
 };
 
+static int iommu_superpage_capability(struct intel_iommu *iommu, bool first_stage)
+{
+	if (!intel_iommu_superpage)
+		return 0;
+
+	if (first_stage)
+		return cap_fl1gp_support(iommu->cap) ? 2 : 1;
+
+	return fls(cap_super_page_val(iommu->cap));
+}
+
+static struct dmar_domain *paging_domain_alloc(struct device *dev, bool first_stage)
+{
+	struct device_domain_info *info = dev_iommu_priv_get(dev);
+	struct intel_iommu *iommu = info->iommu;
+	struct dmar_domain *domain;
+	int addr_width;
+
+	domain = kzalloc(sizeof(*domain), GFP_KERNEL);
+	if (!domain)
+		return ERR_PTR(-ENOMEM);
+
+	INIT_LIST_HEAD(&domain->devices);
+	INIT_LIST_HEAD(&domain->dev_pasids);
+	INIT_LIST_HEAD(&domain->cache_tags);
+	spin_lock_init(&domain->lock);
+	spin_lock_init(&domain->cache_lock);
+	xa_init(&domain->iommu_array);
+
+	domain->nid = dev_to_node(dev);
+	domain->has_iotlb_device = info->ats_enabled;
+	domain->use_first_level = first_stage;
+
+	/* calculate the address width */
+	addr_width = agaw_to_width(iommu->agaw);
+	if (addr_width > cap_mgaw(iommu->cap))
+		addr_width = cap_mgaw(iommu->cap);
+	domain->gaw = addr_width;
+	domain->agaw = iommu->agaw;
+	domain->max_addr = __DOMAIN_MAX_ADDR(addr_width);
+
+	/* iommu memory access coherency */
+	domain->iommu_coherency = iommu_paging_structure_coherency(iommu);
+
+	/* pagesize bitmap */
+	domain->domain.pgsize_bitmap = SZ_4K;
+	domain->iommu_superpage = iommu_superpage_capability(iommu, first_stage);
+	domain->domain.pgsize_bitmap |= domain_super_pgsize_bitmap(domain);
+
+	/*
+	 * IOVA aperture: First-level translation restricts the input-address
+	 * to a canonical address (i.e., address bits 63:N have the same value
+	 * as address bit [N-1], where N is 48-bits with 4-level paging and
+	 * 57-bits with 5-level paging). Hence, skip bit [N-1].
+	 */
+	domain->domain.geometry.force_aperture = true;
+	domain->domain.geometry.aperture_start = 0;
+	if (first_stage)
+		domain->domain.geometry.aperture_end = __DOMAIN_MAX_ADDR(domain->gaw - 1);
+	else
+		domain->domain.geometry.aperture_end = __DOMAIN_MAX_ADDR(domain->gaw);
+
+	/* always allocate the top pgd */
+	domain->pgd = iommu_alloc_page_node(domain->nid, GFP_KERNEL);
+	if (!domain->pgd) {
+		kfree(domain);
+		return ERR_PTR(-ENOMEM);
+	}
+	domain_flush_cache(domain, domain->pgd, PAGE_SIZE);
+
+	return domain;
+}
+
 static struct iommu_domain *intel_iommu_domain_alloc(unsigned type)
 {
 	struct dmar_domain *dmar_domain;
@@ -3695,15 +3768,11 @@ intel_iommu_domain_alloc_user(struct device *dev, u32 flags,
 	if (user_data || (dirty_tracking && !ssads_supported(iommu)))
 		return ERR_PTR(-EOPNOTSUPP);
 
-	/*
-	 * domain_alloc_user op needs to fully initialize a domain before
-	 * return, so uses iommu_domain_alloc() here for simple.
-	 */
-	domain = iommu_domain_alloc(dev->bus);
-	if (!domain)
-		return ERR_PTR(-ENOMEM);
-
-	dmar_domain = to_dmar_domain(domain);
+	/* Do not use first stage for user domain translation. */
+	dmar_domain = paging_domain_alloc(dev, false);
+	if (IS_ERR(dmar_domain))
+		return ERR_CAST(dmar_domain);
+	domain = &dmar_domain->domain;
 
 	if (nested_parent) {
 		dmar_domain->nested_parent = true;
-- 
2.34.1



  parent reply	other threads:[~2024-06-10  9:00 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-10  8:55 [PATCH v3 00/21] iommu: Refactoring domain allocation interface Lu Baolu
2024-06-10  8:55 ` [PATCH v3 01/21] iommu: Add iommu_paging_domain_alloc() interface Lu Baolu
2024-06-19 14:20   ` Jason Gunthorpe
2024-06-27  7:10   ` Vasant Hegde
2024-06-10  8:55 ` [PATCH v3 02/21] iommufd: Use iommu_paging_domain_alloc() Lu Baolu
2024-06-19 14:21   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 03/21] vfio/type1: " Lu Baolu
2024-06-19 14:21   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 04/21] vhost-vdpa: " Lu Baolu
2024-06-19 14:27   ` Jason Gunthorpe
2024-07-03 16:32   ` Michael S. Tsirkin
2024-07-04 13:53     ` Will Deacon
2024-06-10  8:55 ` [PATCH v3 05/21] drm/msm: " Lu Baolu
2024-06-10  8:55 ` [PATCH v3 06/21] drm/nouveau/tegra: " Lu Baolu
2024-06-10  8:55 ` [PATCH v3 07/21] gpu: host1x: " Lu Baolu
2024-06-19 14:30   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 08/21] media: nvidia: tegra: " Lu Baolu
2024-06-19 14:31   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 09/21] media: venus: firmware: " Lu Baolu
2024-06-19 14:32   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 10/21] wifi: ath10k: " Lu Baolu
2024-06-19 14:37   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 11/21] wifi: ath11k: " Lu Baolu
2024-06-19 14:38   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 12/21] remoteproc: " Lu Baolu
2024-06-19 14:38   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 13/21] soc/fsl/qbman: " Lu Baolu
2024-06-19 14:39   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 14/21] RDMA/usnic: " Lu Baolu
2024-06-10  8:55 ` Lu Baolu [this message]
2024-06-19 15:12   ` [PATCH v3 15/21] iommu/vt-d: Add helper to allocate paging domain Jason Gunthorpe
2024-06-20  1:14     ` Baolu Lu
2024-06-28  5:42   ` Yi Liu
2024-06-28  8:29     ` Baolu Lu
2024-06-10  8:55 ` [PATCH v3 16/21] ARM: dma-mapping: Pass device to arm_iommu_create_mapping() Lu Baolu
2024-06-10  8:55 ` [PATCH v3 17/21] ARM: dma-mapping: Use iommu_paging_domain_alloc() Lu Baolu
2024-06-19 15:13   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 18/21] drm/rockchip: " Lu Baolu
2024-06-19 15:14   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 19/21] drm/tegra: Remove call to iommu_domain_alloc() Lu Baolu
2024-06-10  8:55 ` [PATCH v3 20/21] iommu: Remove iommu_present() Lu Baolu
2024-06-19 15:14   ` Jason Gunthorpe
2024-06-10  8:55 ` [PATCH v3 21/21] iommu: Remove iommu_domain_alloc() Lu Baolu
2024-06-19 15:14   ` Jason Gunthorpe
2024-06-19 14:37 ` [PATCH v3 00/21] iommu: Refactoring domain allocation interface Jason Gunthorpe
2024-07-04 14:18 ` Will Deacon
2024-07-04 14:24   ` Baolu Lu
2024-07-04 14:34     ` Will Deacon
2024-07-08 16:34   ` Jason Gunthorpe
2024-07-09  2:10     ` Baolu Lu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240610085555.88197-16-baolu.lu@linux.intel.com \
    --to=baolu.lu@linux.intel.com \
    --cc=airlied@gmail.com \
    --cc=alex.williamson@redhat.com \
    --cc=andersson@kernel.org \
    --cc=ath10k@lists.infradead.org \
    --cc=ath11k@lists.infradead.org \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=iommu@lists.linux.dev \
    --cc=jasowang@redhat.com \
    --cc=jgg@ziepe.ca \
    --cc=jonathanh@nvidia.com \
    --cc=joro@8bytes.org \
    --cc=kevin.tian@intel.com \
    --cc=kvalo@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mathieu.poirier@linaro.org \
    --cc=mperttunen@nvidia.com \
    --cc=mst@redhat.com \
    --cc=quic_jjohnson@quicinc.com \
    --cc=robin.murphy@arm.com \
    --cc=thierry.reding@gmail.com \
    --cc=will@kernel.org \
    --cc=yi.l.liu@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.