Re: [PATCH v2 04/10] iommu/dma: Use the gfp parameter in __iommu_dma_alloc_noncontiguous()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]<

To: Jason Gunthorpe <jgg@xxxxxxxxxx>, Lu Baolu <baolu.lu@xxxxxxxxxxxxxxx>, Joerg Roedel <joro@xxxxxxxxxx>, Matthew Rosato <mjrosato@xxxxxxxxxxxxx>, Robin Murphy <robin.murphy@xxxxxxx>
Subject: Re: [PATCH v2 04/10] iommu/dma: Use the gfp parameter in __iommu_dma_alloc_noncontiguous()
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Thu, 19 Jan 2023 03:47:19 +0000
Accept-language: en-US
Cc: "linux-s390@xxxxxxxxxxxxxxx" <linux-s390@xxxxxxxxxxxxxxx>, "kvm@xxxxxxxxxxxxxxx" <kvm@xxxxxxxxxxxxxxx>, "nouveau@xxxxxxxxxxxxxxxxxxxxx" <nouveau@xxxxxxxxxxxxxxxxxxxxx>, "linux-rdma@xxxxxxxxxxxxxxx" <linux-rdma@xxxxxxxxxxxxxxx>, "linux-arm-msm@xxxxxxxxxxxxxxx" <linux-arm-msm@xxxxxxxxxxxxxxx>, Niklas Schnelle <schnelle@xxxxxxxxxxxxx>, "linux-remoteproc@xxxxxxxxxxxxxxx" <linux-remoteproc@xxxxxxxxxxxxxxx>, "iommu@xxxxxxxxxxxxxxx" <iommu@xxxxxxxxxxxxxxx>, "dri-devel@xxxxxxxxxxxxxxxxxxxxx" <dri-devel@xxxxxxxxxxxxxxxxxxxxx>, "linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx" <linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>, Alex Williamson <alex.williamson@xxxxxxxxxx>, "netdev@xxxxxxxxxxxxxxx" <netdev@xxxxxxxxxxxxxxx>, "ath10k@xxxxxxxxxxxxxxxxxxx" <ath10k@xxxxxxxxxxxxxxxxxxx>, "linux-wireless@xxxxxxxxxxxxxxx" <linux-wireless@xxxxxxxxxxxxxxx>, "linux-tegra@xxxxxxxxxxxxxxx" <linux-tegra@xxxxxxxxxxxxxxx>, Christian Borntraeger <borntraeger@xxxxxxxxxxxxx>, "virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx" <virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx>, "ath11k@xxxxxxxxxxxxxxxxxxx" <ath11k@xxxxxxxxxxxxxxxxxxx>, "linux-arm-kernel@xxxxxxxxxxxxxxxxxxx" <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>, "linux-media@xxxxxxxxxxxxxxx" <linux-media@xxxxxxxxxxxxxxx>
In-reply-to: <4-v2-ce66f632bd0d+484-iommu_map_gfp_jgg@nvidia.com>
References: <0-v2-ce66f632bd0d+484-iommu_map_gfp_jgg@nvidia.com> <4-v2-ce66f632bd0d+484-iommu_map_gfp_jgg@nvidia.com>

> From: Jason Gunthorpe <jgg@xxxxxxxxxx>
> Sent: Thursday, January 19, 2023 2:01 AM
> 
> Change the sg_alloc_table_from_pages() allocation that was hardwired to
> GFP_KERNEL to use the gfp parameter like the other allocations in this
> function.
> 
> Auditing says this is never called from an atomic context, so it is safe
> as is, but reads wrong.
> 
> Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxx>

Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>

References:
- [PATCH v2 00/10] Let iommufd charge IOPTE allocations to the memory cgroup
  - From: Jason Gunthorpe
- [PATCH v2 04/10] iommu/dma: Use the gfp parameter in __iommu_dma_alloc_noncontiguous()
  - From: Jason Gunthorpe

Prev by Date: Re: [Intel-gfx] [PATCH 1/2] drm/ttm: prevent moving of pinned BOs
Next by Date: Re: [PATCH v2 07/10] iommu/intel: Support the gfp argument to the map_pages op
Previous by thread: Re: [PATCH v2 04/10] iommu/dma: Use the gfp parameter in __iommu_dma_alloc_noncontiguous()
Next by thread: [PATCH v2 01/10] iommu: Add a gfp parameter to iommu_map()
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Security] [Bugtraq] [Linux] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]