All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/8] Let iommufd charge IOPTE allocations to the memory cgroup
@ 2023-01-06 16:42 ` Jason Gunthorpe
  0 siblings, 0 replies; 144+ messages in thread
From: Jason Gunthorpe @ 2023-01-06 16:42 UTC (permalink / raw)
  To: Lu Baolu, Joerg Roedel, Kevin Tian, Matthew Rosato, Robin Murphy
  Cc: Alex Williamson, ath10k, ath11k, Christian Borntraeger, dri-devel,
	iommu, kvm, linux-arm-kernel, linux-arm-msm, linux-media,
	linux-rdma, linux-remoteproc, linux-s390, linux-stm32,
	linux-tegra, linux-wireless, netdev, nouveau, Niklas Schnelle,
	virtualization

iommufd follows the same design as KVM and uses memory cgroups to limit
the amount of kernel memory a iommufd file descriptor can pin down. The
various internal data structures already use GFP_KERNEL_ACCOUNT to charge
its own memory.

However, one of the biggest consumers of kernel memory is the IOPTEs
stored under the iommu_domain and these allocations are not tracked.

This series is the first step in fixing it.

The iommu driver contract already includes a 'gfp' argument to the
map_pages op, allowing iommufd to specify GFP_KERNEL_ACCOUNT and then
having the driver allocate the IOPTE tables with that flag will capture a
significant amount of the allocations.

Update the iommu_map() API to pass in the GFP argument, and fix all call
sites. Replace iommu_map_atomic().

Audit the "enterprise" iommu drivers to make sure they do the right thing.
Intel and S390 ignore the GFP argument and always use GFP_ATOMIC. This is
problematic for iommufd anyhow, so fix it. AMD and ARM SMMUv2/3 are
already correct.

A follow up series will be needed to capture the allocations made when the
iommu_domain itself is allocated, which will complete the job.

Jason Gunthorpe (8):
  iommu: Add a gfp parameter to iommu_map()
  iommu: Remove iommu_map_atomic()
  iommu: Add a gfp parameter to iommu_map_sg()
  iommu/dma: Use the gfp parameter in __iommu_dma_alloc_noncontiguous()
  iommufd: Use GFP_KERNEL_ACCOUNT for iommu_map()
  iommu/intel: Add a gfp parameter to alloc_pgtable_page()
  iommu/intel: Support the gfp argument to the map_pages op
  iommu/s390: Push the gfp parameter to the kmem_cache_alloc()'s

 arch/arm/mm/dma-mapping.c                     | 11 +++--
 arch/s390/include/asm/pci_dma.h               |  5 ++-
 arch/s390/pci/pci_dma.c                       | 31 +++++++------
 .../drm/nouveau/nvkm/subdev/instmem/gk20a.c   |  3 +-
 drivers/gpu/drm/tegra/drm.c                   |  2 +-
 drivers/gpu/host1x/cdma.c                     |  2 +-
 drivers/infiniband/hw/usnic/usnic_uiom.c      |  4 +-
 drivers/iommu/dma-iommu.c                     | 11 ++---
 drivers/iommu/intel/iommu.c                   | 36 +++++++++-------
 drivers/iommu/intel/iommu.h                   |  2 +-
 drivers/iommu/intel/pasid.c                   |  2 +-
 drivers/iommu/iommu.c                         | 43 +++++--------------
 drivers/iommu/iommufd/pages.c                 |  6 ++-
 drivers/iommu/s390-iommu.c                    | 15 ++++---
 drivers/media/platform/qcom/venus/firmware.c  |  2 +-
 drivers/net/ipa/ipa_mem.c                     |  6 ++-
 drivers/net/wireless/ath/ath10k/snoc.c        |  2 +-
 drivers/net/wireless/ath/ath11k/ahb.c         |  4 +-
 drivers/remoteproc/remoteproc_core.c          |  5 ++-
 drivers/vfio/vfio_iommu_type1.c               |  9 ++--
 drivers/vhost/vdpa.c                          |  2 +-
 include/linux/iommu.h                         | 31 +++----------
 22 files changed, 109 insertions(+), 125 deletions(-)


base-commit: 88603b6dc419445847923fcb7fe5080067a30f98
-- 
2.39.0


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 144+ messages in thread

end of thread, other threads:[~2023-05-04 12:32 UTC | newest]

Thread overview: 144+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-06 16:42 [PATCH 0/8] Let iommufd charge IOPTE allocations to the memory cgroup Jason Gunthorpe
2023-01-06 16:42 ` Jason Gunthorpe
2023-01-06 16:42 ` Jason Gunthorpe
2023-01-06 16:42 ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42 ` Jason Gunthorpe
2023-01-06 16:42 ` Jason Gunthorpe
2023-01-06 16:42 ` [PATCH 1/8] iommu: Add a gfp parameter to iommu_map() Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 17:15   ` Robin Murphy
2023-01-06 17:15     ` Robin Murphy
2023-01-06 17:15     ` Robin Murphy
2023-01-06 17:15     ` Robin Murphy
2023-01-06 17:15     ` [Nouveau] " Robin Murphy
2023-01-06 17:15     ` Robin Murphy
2023-01-06 17:15     ` Robin Murphy
2023-01-06 17:24     ` Jason Gunthorpe
2023-01-06 17:24       ` Jason Gunthorpe
2023-01-06 17:24       ` Jason Gunthorpe
2023-01-06 17:24       ` [Nouveau] " Jason Gunthorpe
2023-01-06 17:24       ` Jason Gunthorpe
2023-01-06 17:24       ` Jason Gunthorpe
2023-01-20  9:24       ` Joerg Roedel
2023-01-20  9:24         ` Joerg Roedel
2023-01-20  9:24         ` Joerg Roedel
2023-01-20  9:24         ` Joerg Roedel
2023-01-20  9:24         ` [Nouveau] " Joerg Roedel
2023-01-20  9:24         ` Joerg Roedel
2023-01-20  9:24         ` Joerg Roedel
2023-01-20 17:53         ` Jason Gunthorpe
2023-01-20 17:53           ` Jason Gunthorpe
2023-01-20 17:53           ` Jason Gunthorpe
2023-01-20 17:53           ` [Nouveau] " Jason Gunthorpe
2023-01-20 17:53           ` Jason Gunthorpe
2023-01-20 17:53           ` Jason Gunthorpe
2023-01-23  9:59           ` Joerg Roedel
2023-01-23  9:59             ` Joerg Roedel
2023-01-23  9:59             ` Joerg Roedel
2023-01-23  9:59             ` Joerg Roedel
2023-01-23  9:59             ` [Nouveau] " Joerg Roedel
2023-01-23  9:59             ` Joerg Roedel
2023-01-23  9:59             ` Joerg Roedel
2023-01-16 18:12     ` Jason Gunthorpe
2023-01-16 18:12       ` Jason Gunthorpe
2023-01-16 18:12       ` Jason Gunthorpe
2023-01-16 18:12       ` [Nouveau] " Jason Gunthorpe
2023-01-16 18:12       ` Jason Gunthorpe
2023-01-16 18:12       ` Jason Gunthorpe
2023-01-06 16:42 ` [PATCH 2/8] iommu: Remove iommu_map_atomic() Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42 ` [PATCH 3/8] iommu: Add a gfp parameter to iommu_map_sg() Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42 ` [PATCH 4/8] iommu/dma: Use the gfp parameter in __iommu_dma_alloc_noncontiguous() Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42 ` [PATCH 5/8] iommufd: Use GFP_KERNEL_ACCOUNT for iommu_map() Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42 ` [PATCH 6/8] iommu/intel: Add a gfp parameter to alloc_pgtable_page() Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-17  3:35   ` Tian, Kevin
2023-01-17  3:35     ` Tian, Kevin
2023-01-17  3:35     ` Tian, Kevin
2023-01-17  3:35     ` Tian, Kevin
2023-01-17  3:35     ` [Nouveau] " Tian, Kevin
2023-01-17  3:35     ` Tian, Kevin
2023-01-17  3:35     ` Tian, Kevin
2023-01-17 13:30     ` Jason Gunthorpe
2023-01-17 13:30       ` Jason Gunthorpe
2023-01-17 13:30       ` Jason Gunthorpe
2023-01-17 13:30       ` [Nouveau] " Jason Gunthorpe
2023-01-17 13:30       ` Jason Gunthorpe
2023-01-17 13:30       ` Jason Gunthorpe
2023-01-18  1:18       ` Tian, Kevin
2023-01-18  1:18         ` Tian, Kevin
2023-01-18  1:18         ` Tian, Kevin
2023-01-18  1:18         ` Tian, Kevin
2023-01-18  1:18         ` [Nouveau] " Tian, Kevin
2023-01-18  1:18         ` Tian, Kevin
2023-01-18  1:18         ` Tian, Kevin
2023-01-18 15:15         ` Jason Gunthorpe
2023-01-18 15:15           ` Jason Gunthorpe
2023-01-18 15:15           ` Jason Gunthorpe
2023-01-18 15:15           ` [Nouveau] " Jason Gunthorpe
2023-01-18 15:15           ` Jason Gunthorpe
2023-01-18 15:15           ` Jason Gunthorpe
2023-01-06 16:42 ` [PATCH 7/8] iommu/intel: Support the gfp argument to the map_pages op Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-17  3:38   ` Tian, Kevin
2023-01-17  3:38     ` Tian, Kevin
2023-01-17  3:38     ` Tian, Kevin
2023-01-17  3:38     ` Tian, Kevin
2023-01-17  3:38     ` [Nouveau] " Tian, Kevin
2023-01-17  3:38     ` Tian, Kevin
2023-01-17  3:38     ` Tian, Kevin
2023-01-17  8:35     ` Baolu Lu
2023-01-17  8:35       ` Baolu Lu
2023-01-17  8:35       ` Baolu Lu
2023-01-17  8:35       ` [Nouveau] " Baolu Lu
2023-01-17  8:35       ` Baolu Lu
2023-01-17  8:35       ` Baolu Lu
2023-01-17 13:28     ` Jason Gunthorpe
2023-01-17 13:28       ` Jason Gunthorpe
2023-01-17 13:28       ` Jason Gunthorpe
2023-01-17 13:28       ` [Nouveau] " Jason Gunthorpe
2023-01-17 13:28       ` Jason Gunthorpe
2023-01-17 13:28       ` Jason Gunthorpe
2023-01-06 16:42 ` [PATCH 8/8] iommu/s390: Push the gfp parameter to the kmem_cache_alloc()'s Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` [Nouveau] " Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-06 16:42   ` Jason Gunthorpe
2023-01-17  8:49   ` Niklas Schnelle
2023-01-17  8:49     ` Niklas Schnelle
2023-01-17  8:49     ` Niklas Schnelle
2023-01-17  8:49     ` [Nouveau] " Niklas Schnelle
2023-01-17  8:49     ` Niklas Schnelle
2023-01-17  8:49     ` Niklas Schnelle

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.