iommu/amd: Do not BUG_ON in __detach_device()
authorJoerg Roedel <jroedel@suse.de>
Tue, 20 Oct 2015 15:33:34 +0000 (17:33 +0200)
committerJoerg Roedel <jroedel@suse.de>
Wed, 21 Oct 2015 09:29:26 +0000 (11:29 +0200)
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>
drivers/iommu/amd_iommu.c

index f82060e..6070b15 100644 (file)
@@ -2204,7 +2204,8 @@ static void __detach_device(struct iommu_dev_data *dev_data)
        struct protection_domain *domain;
        unsigned long flags;
 
-       BUG_ON(!dev_data->domain);
+       if (WARN_ON(!dev_data->domain))
+               return;
 
        domain = dev_data->domain;