diff options
| author | Bjorn Helgaas <bhelgaas@google.com> | 2019-02-08 16:06:22 -0600 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2019-02-11 12:07:06 +0100 |
| commit | f096d6657a7714e62a4799893ac9bdee75be29c8 (patch) | |
| tree | 852435cf03cf221eed5a4c99cf948035705a05b2 /drivers/iommu | |
| parent | 717532394c712e044195405685f62522c579e22a (diff) | |
| download | linux-f096d6657a7714e62a4799893ac9bdee75be29c8.tar.bz2 | |
iommu/vt-d: Remove misleading "domain 0" test from domain_exit()
The "Domain 0 is reserved, so dont process it" comment suggests that a NULL
pointer corresponds to domain 0. I don't think that's true, and in any
case, every caller supplies a non-NULL domain pointer that has already been
dereferenced, so the test is unnecessary.
Remove the test for a null "domain" pointer. No functional change
intended.
This null pointer check was added by 5e98c4b1d6e8 ("Allocation and free
functions of virtual machine domain").
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu')
| -rw-r--r-- | drivers/iommu/intel-iommu.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c index e06913460cc3..271455e5026e 100644 --- a/drivers/iommu/intel-iommu.c +++ b/drivers/iommu/intel-iommu.c @@ -1903,10 +1903,6 @@ static void domain_exit(struct dmar_domain *domain) { struct page *freelist; - /* Domain 0 is reserved, so dont process it */ - if (!domain) - return; - /* Remove associated devices and clear attached or cached domains */ rcu_read_lock(); domain_remove_dev_info(domain); |