diff options
author | Joerg Roedel <joerg.roedel@amd.com> | 2008-12-10 14:05:52 -0500 |
---|---|---|
committer | Joerg Roedel <joerg.roedel@amd.com> | 2009-01-03 08:11:54 -0500 |
commit | 9e919012e33c481991e46aa4cb13d807cd47b798 (patch) | |
tree | cba1c652c818ed0618c265a5e969a37707ce8fe2 | |
parent | 237b6f33291394c337ae84e2d3782d5605803af2 (diff) |
AMD IOMMU: don't remove protection domain from iommu_pd_list
Impact: save unneeded logic to add and remove domains to the list
The removal of a protection domain from the iommu_pd_list is not
necessary. Another benefit is that we save complexity because we don't
have to readd it later when the device no longer uses the domain.
Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
-rw-r--r-- | arch/x86/kernel/amd_iommu.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/arch/x86/kernel/amd_iommu.c b/arch/x86/kernel/amd_iommu.c index fee16fbf2f33..b7b3067630cf 100644 --- a/arch/x86/kernel/amd_iommu.c +++ b/arch/x86/kernel/amd_iommu.c | |||
@@ -844,7 +844,6 @@ static struct dma_ops_domain *find_protection_domain(u16 devid) | |||
844 | list_for_each_entry(entry, &iommu_pd_list, list) { | 844 | list_for_each_entry(entry, &iommu_pd_list, list) { |
845 | if (entry->target_dev == devid) { | 845 | if (entry->target_dev == devid) { |
846 | ret = entry; | 846 | ret = entry; |
847 | list_del(&ret->list); | ||
848 | break; | 847 | break; |
849 | } | 848 | } |
850 | } | 849 | } |