diff options
author | Joerg Roedel <jroedel@suse.de> | 2015-10-20 11:33:34 -0400 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2015-10-21 05:29:26 -0400 |
commit | f34c73f55a06ac443c492fc11b85f7a44dfdc112 (patch) | |
tree | ebe71ad583fb74d04d65e9c85af2a6d2d724cb5c /drivers/iommu | |
parent | a591989a7c162587f24305c3fe3bd8f055ed3329 (diff) |
iommu/amd: Do not BUG_ON in __detach_device()
The condition in the BUG_ON is an indicator of a BUG, but no
reason to kill the code path. Turn it into a WARN_ON and
bail out if it is hit.
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu')
-rw-r--r-- | drivers/iommu/amd_iommu.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/iommu/amd_iommu.c b/drivers/iommu/amd_iommu.c index f82060e778a2..6070b1504497 100644 --- a/drivers/iommu/amd_iommu.c +++ b/drivers/iommu/amd_iommu.c | |||
@@ -2204,7 +2204,8 @@ static void __detach_device(struct iommu_dev_data *dev_data) | |||
2204 | struct protection_domain *domain; | 2204 | struct protection_domain *domain; |
2205 | unsigned long flags; | 2205 | unsigned long flags; |
2206 | 2206 | ||
2207 | BUG_ON(!dev_data->domain); | 2207 | if (WARN_ON(!dev_data->domain)) |
2208 | return; | ||
2208 | 2209 | ||
2209 | domain = dev_data->domain; | 2210 | domain = dev_data->domain; |
2210 | 2211 | ||