Commit 8deebd2d authored by Joerg Roedel's avatar Joerg Roedel Committed by Greg Kroah-Hartman

iommu/amd: Handle IOMMU_DOMAIN_DMA in ops->domain_free call-back

commit cda7005b upstream.

This domain type is not yet handled in the
iommu_ops->domain_free() call-back. Fix that.

Fixes: 0bb6e243 ('iommu/amd: Support IOMMU_DOMAIN_DMA type allocation')
Signed-off-by: default avatarJoerg Roedel <jroedel@suse.de>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 3bc03c3c
...@@ -3103,9 +3103,7 @@ static struct iommu_domain *amd_iommu_domain_alloc(unsigned type) ...@@ -3103,9 +3103,7 @@ static struct iommu_domain *amd_iommu_domain_alloc(unsigned type)
static void amd_iommu_domain_free(struct iommu_domain *dom) static void amd_iommu_domain_free(struct iommu_domain *dom)
{ {
struct protection_domain *domain; struct protection_domain *domain;
struct dma_ops_domain *dma_dom;
if (!dom)
return;
domain = to_pdomain(dom); domain = to_pdomain(dom);
...@@ -3114,13 +3112,24 @@ static void amd_iommu_domain_free(struct iommu_domain *dom) ...@@ -3114,13 +3112,24 @@ static void amd_iommu_domain_free(struct iommu_domain *dom)
BUG_ON(domain->dev_cnt != 0); BUG_ON(domain->dev_cnt != 0);
if (domain->mode != PAGE_MODE_NONE) if (!dom)
free_pagetable(domain); return;
if (domain->flags & PD_IOMMUV2_MASK) switch (dom->type) {
free_gcr3_table(domain); case IOMMU_DOMAIN_DMA:
dma_dom = domain->priv;
dma_ops_domain_free(dma_dom);
break;
default:
if (domain->mode != PAGE_MODE_NONE)
free_pagetable(domain);
protection_domain_free(domain); if (domain->flags & PD_IOMMUV2_MASK)
free_gcr3_table(domain);
protection_domain_free(domain);
break;
}
} }
static void amd_iommu_detach_device(struct iommu_domain *dom, static void amd_iommu_detach_device(struct iommu_domain *dom,
......
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