diff options
| author |   Joerg Roedel <jroedel@suse.de> | 2015-10-20 17:33:36 +0200 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2015-10-21 11:29:27 +0200 |
| commit | f1dd0a8bcd67e34537fe02fa9cda15aa46368d11 (patch) | |
| tree | e9971be56dd2bec7f83fd9ffa83407c23cbe5673 /drivers/iommu | |
| parent | iommu/amd: Do not iterate over alias-list in __[attach|detach]_device (diff) | |
| download | linux-dev-f1dd0a8bcd67e34537fe02fa9cda15aa46368d11.tar.xz linux-dev-f1dd0a8bcd67e34537fe02fa9cda15aa46368d11.zip | |
iommu/amd: Don't disable IRQs in __detach_device
This function is already called with IRQs disabled already.
So no need to disable them again.
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu')
| -rw-r--r-- | drivers/iommu/amd_iommu.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/iommu/amd_iommu.c b/drivers/iommu/amd_iommu.c index e8443e6ff1b1..07f491c2dc64 100644 --- a/drivers/iommu/amd_iommu.c +++ b/drivers/iommu/amd_iommu.c @@ -2189,18 +2189,17 @@ static int attach_device(struct device *dev, static void __detach_device(struct iommu_dev_data *dev_data) { struct protection_domain *domain; - unsigned long flags; if (WARN_ON(!dev_data->domain)) return; domain = dev_data->domain; - spin_lock_irqsave(&domain->lock, flags); + spin_lock(&domain->lock); do_detach(dev_data); - spin_unlock_irqrestore(&domain->lock, flags); + spin_unlock(&domain->lock); } /* |