Commit 0251d010 authored by Lu Baolu's avatar Lu Baolu Committed by Joerg Roedel

iommu: Add gfp parameter to iommu_alloc_resv_region

Add gfp parameter to iommu_alloc_resv_region() for the callers to specify
the memory allocation behavior. Thus iommu_alloc_resv_region() could also
be available in critical contexts.
Signed-off-by: default avatarLu Baolu <baolu.lu@linux.intel.com>
Tested-by: default avatarAlex Williamson <alex.williamson@redhat.com>
Link: https://lore.kernel.org/r/20220927053109.4053662-2-baolu.lu@linux.intel.comSigned-off-by: default avatarJoerg Roedel <jroedel@suse.de>
parent 9abf2313
...@@ -1142,7 +1142,8 @@ static void iort_iommu_msi_get_resv_regions(struct device *dev, ...@@ -1142,7 +1142,8 @@ static void iort_iommu_msi_get_resv_regions(struct device *dev,
struct iommu_resv_region *region; struct iommu_resv_region *region;
region = iommu_alloc_resv_region(base + SZ_64K, SZ_64K, region = iommu_alloc_resv_region(base + SZ_64K, SZ_64K,
prot, IOMMU_RESV_MSI); prot, IOMMU_RESV_MSI,
GFP_KERNEL);
if (region) if (region)
list_add_tail(&region->list, head); list_add_tail(&region->list, head);
} }
......
...@@ -2330,7 +2330,8 @@ static void amd_iommu_get_resv_regions(struct device *dev, ...@@ -2330,7 +2330,8 @@ static void amd_iommu_get_resv_regions(struct device *dev,
type = IOMMU_RESV_RESERVED; type = IOMMU_RESV_RESERVED;
region = iommu_alloc_resv_region(entry->address_start, region = iommu_alloc_resv_region(entry->address_start,
length, prot, type); length, prot, type,
GFP_KERNEL);
if (!region) { if (!region) {
dev_err(dev, "Out of memory allocating dm-regions\n"); dev_err(dev, "Out of memory allocating dm-regions\n");
return; return;
...@@ -2340,14 +2341,14 @@ static void amd_iommu_get_resv_regions(struct device *dev, ...@@ -2340,14 +2341,14 @@ static void amd_iommu_get_resv_regions(struct device *dev,
region = iommu_alloc_resv_region(MSI_RANGE_START, region = iommu_alloc_resv_region(MSI_RANGE_START,
MSI_RANGE_END - MSI_RANGE_START + 1, MSI_RANGE_END - MSI_RANGE_START + 1,
0, IOMMU_RESV_MSI); 0, IOMMU_RESV_MSI, GFP_KERNEL);
if (!region) if (!region)
return; return;
list_add_tail(&region->list, head); list_add_tail(&region->list, head);
region = iommu_alloc_resv_region(HT_RANGE_START, region = iommu_alloc_resv_region(HT_RANGE_START,
HT_RANGE_END - HT_RANGE_START + 1, HT_RANGE_END - HT_RANGE_START + 1,
0, IOMMU_RESV_RESERVED); 0, IOMMU_RESV_RESERVED, GFP_KERNEL);
if (!region) if (!region)
return; return;
list_add_tail(&region->list, head); list_add_tail(&region->list, head);
......
...@@ -758,7 +758,7 @@ static void apple_dart_get_resv_regions(struct device *dev, ...@@ -758,7 +758,7 @@ static void apple_dart_get_resv_regions(struct device *dev,
region = iommu_alloc_resv_region(DOORBELL_ADDR, region = iommu_alloc_resv_region(DOORBELL_ADDR,
PAGE_SIZE, prot, PAGE_SIZE, prot,
IOMMU_RESV_MSI); IOMMU_RESV_MSI, GFP_KERNEL);
if (!region) if (!region)
return; return;
......
...@@ -2757,7 +2757,7 @@ static void arm_smmu_get_resv_regions(struct device *dev, ...@@ -2757,7 +2757,7 @@ static void arm_smmu_get_resv_regions(struct device *dev,
int prot = IOMMU_WRITE | IOMMU_NOEXEC | IOMMU_MMIO; int prot = IOMMU_WRITE | IOMMU_NOEXEC | IOMMU_MMIO;
region = iommu_alloc_resv_region(MSI_IOVA_BASE, MSI_IOVA_LENGTH, region = iommu_alloc_resv_region(MSI_IOVA_BASE, MSI_IOVA_LENGTH,
prot, IOMMU_RESV_SW_MSI); prot, IOMMU_RESV_SW_MSI, GFP_KERNEL);
if (!region) if (!region)
return; return;
......
...@@ -1534,7 +1534,7 @@ static void arm_smmu_get_resv_regions(struct device *dev, ...@@ -1534,7 +1534,7 @@ static void arm_smmu_get_resv_regions(struct device *dev,
int prot = IOMMU_WRITE | IOMMU_NOEXEC | IOMMU_MMIO; int prot = IOMMU_WRITE | IOMMU_NOEXEC | IOMMU_MMIO;
region = iommu_alloc_resv_region(MSI_IOVA_BASE, MSI_IOVA_LENGTH, region = iommu_alloc_resv_region(MSI_IOVA_BASE, MSI_IOVA_LENGTH,
prot, IOMMU_RESV_SW_MSI); prot, IOMMU_RESV_SW_MSI, GFP_KERNEL);
if (!region) if (!region)
return; return;
......
...@@ -4552,7 +4552,8 @@ static void intel_iommu_get_resv_regions(struct device *device, ...@@ -4552,7 +4552,8 @@ static void intel_iommu_get_resv_regions(struct device *device,
IOMMU_RESV_DIRECT_RELAXABLE : IOMMU_RESV_DIRECT; IOMMU_RESV_DIRECT_RELAXABLE : IOMMU_RESV_DIRECT;
resv = iommu_alloc_resv_region(rmrr->base_address, resv = iommu_alloc_resv_region(rmrr->base_address,
length, prot, type); length, prot, type,
GFP_KERNEL);
if (!resv) if (!resv)
break; break;
...@@ -4567,7 +4568,8 @@ static void intel_iommu_get_resv_regions(struct device *device, ...@@ -4567,7 +4568,8 @@ static void intel_iommu_get_resv_regions(struct device *device,
if ((pdev->class >> 8) == PCI_CLASS_BRIDGE_ISA) { if ((pdev->class >> 8) == PCI_CLASS_BRIDGE_ISA) {
reg = iommu_alloc_resv_region(0, 1UL << 24, prot, reg = iommu_alloc_resv_region(0, 1UL << 24, prot,
IOMMU_RESV_DIRECT_RELAXABLE); IOMMU_RESV_DIRECT_RELAXABLE,
GFP_KERNEL);
if (reg) if (reg)
list_add_tail(&reg->list, head); list_add_tail(&reg->list, head);
} }
...@@ -4576,7 +4578,7 @@ static void intel_iommu_get_resv_regions(struct device *device, ...@@ -4576,7 +4578,7 @@ static void intel_iommu_get_resv_regions(struct device *device,
reg = iommu_alloc_resv_region(IOAPIC_RANGE_START, reg = iommu_alloc_resv_region(IOAPIC_RANGE_START,
IOAPIC_RANGE_END - IOAPIC_RANGE_START + 1, IOAPIC_RANGE_END - IOAPIC_RANGE_START + 1,
0, IOMMU_RESV_MSI); 0, IOMMU_RESV_MSI, GFP_KERNEL);
if (!reg) if (!reg)
return; return;
list_add_tail(&reg->list, head); list_add_tail(&reg->list, head);
......
...@@ -504,7 +504,7 @@ static int iommu_insert_resv_region(struct iommu_resv_region *new, ...@@ -504,7 +504,7 @@ static int iommu_insert_resv_region(struct iommu_resv_region *new,
LIST_HEAD(stack); LIST_HEAD(stack);
nr = iommu_alloc_resv_region(new->start, new->length, nr = iommu_alloc_resv_region(new->start, new->length,
new->prot, new->type); new->prot, new->type, GFP_KERNEL);
if (!nr) if (!nr)
return -ENOMEM; return -ENOMEM;
...@@ -2579,11 +2579,12 @@ EXPORT_SYMBOL(iommu_put_resv_regions); ...@@ -2579,11 +2579,12 @@ EXPORT_SYMBOL(iommu_put_resv_regions);
struct iommu_resv_region *iommu_alloc_resv_region(phys_addr_t start, struct iommu_resv_region *iommu_alloc_resv_region(phys_addr_t start,
size_t length, int prot, size_t length, int prot,
enum iommu_resv_type type) enum iommu_resv_type type,
gfp_t gfp)
{ {
struct iommu_resv_region *region; struct iommu_resv_region *region;
region = kzalloc(sizeof(*region), GFP_KERNEL); region = kzalloc(sizeof(*region), gfp);
if (!region) if (!region)
return NULL; return NULL;
......
...@@ -917,7 +917,8 @@ static void mtk_iommu_get_resv_regions(struct device *dev, ...@@ -917,7 +917,8 @@ static void mtk_iommu_get_resv_regions(struct device *dev,
continue; continue;
region = iommu_alloc_resv_region(resv->iova_base, resv->size, region = iommu_alloc_resv_region(resv->iova_base, resv->size,
prot, IOMMU_RESV_RESERVED); prot, IOMMU_RESV_RESERVED,
GFP_KERNEL);
if (!region) if (!region)
return; return;
......
...@@ -490,11 +490,13 @@ static int viommu_add_resv_mem(struct viommu_endpoint *vdev, ...@@ -490,11 +490,13 @@ static int viommu_add_resv_mem(struct viommu_endpoint *vdev,
fallthrough; fallthrough;
case VIRTIO_IOMMU_RESV_MEM_T_RESERVED: case VIRTIO_IOMMU_RESV_MEM_T_RESERVED:
region = iommu_alloc_resv_region(start, size, 0, region = iommu_alloc_resv_region(start, size, 0,
IOMMU_RESV_RESERVED); IOMMU_RESV_RESERVED,
GFP_KERNEL);
break; break;
case VIRTIO_IOMMU_RESV_MEM_T_MSI: case VIRTIO_IOMMU_RESV_MEM_T_MSI:
region = iommu_alloc_resv_region(start, size, prot, region = iommu_alloc_resv_region(start, size, prot,
IOMMU_RESV_MSI); IOMMU_RESV_MSI,
GFP_KERNEL);
break; break;
} }
if (!region) if (!region)
...@@ -909,7 +911,8 @@ static void viommu_get_resv_regions(struct device *dev, struct list_head *head) ...@@ -909,7 +911,8 @@ static void viommu_get_resv_regions(struct device *dev, struct list_head *head)
*/ */
if (!msi) { if (!msi) {
msi = iommu_alloc_resv_region(MSI_IOVA_BASE, MSI_IOVA_LENGTH, msi = iommu_alloc_resv_region(MSI_IOVA_BASE, MSI_IOVA_LENGTH,
prot, IOMMU_RESV_SW_MSI); prot, IOMMU_RESV_SW_MSI,
GFP_KERNEL);
if (!msi) if (!msi)
return; return;
......
...@@ -455,7 +455,7 @@ extern void iommu_set_default_translated(bool cmd_line); ...@@ -455,7 +455,7 @@ extern void iommu_set_default_translated(bool cmd_line);
extern bool iommu_default_passthrough(void); extern bool iommu_default_passthrough(void);
extern struct iommu_resv_region * extern struct iommu_resv_region *
iommu_alloc_resv_region(phys_addr_t start, size_t length, int prot, iommu_alloc_resv_region(phys_addr_t start, size_t length, int prot,
enum iommu_resv_type type); enum iommu_resv_type type, gfp_t gfp);
extern int iommu_get_group_resv_regions(struct iommu_group *group, extern int iommu_get_group_resv_regions(struct iommu_group *group,
struct list_head *head); struct list_head *head);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment