iommu/vt-d: Use BUG_ON instead of if () BUG()
authorJoerg Roedel <jroedel@suse.de>
Thu, 13 Aug 2015 09:15:13 +0000 (11:15 +0200)
committerJoerg Roedel <jroedel@suse.de>
Thu, 13 Aug 2015 17:49:46 +0000 (19:49 +0200)
Found by a coccicheck script.

Signed-off-by: Joerg Roedel <jroedel@suse.de>
drivers/iommu/intel-iommu.c

index 8834765e90c68a6736406a7bc26cd2e206b93cb1..2a7e01798f4bea900aa51d96d167bc1556804446 100644 (file)
@@ -4760,8 +4760,7 @@ static size_t intel_iommu_unmap(struct iommu_domain *domain,
 
        /* Cope with horrid API which requires us to unmap more than the
           size argument if it happens to be a large-page mapping. */
-       if (!pfn_to_dma_pte(dmar_domain, iova >> VTD_PAGE_SHIFT, &level))
-               BUG();
+       BUG_ON(!pfn_to_dma_pte(dmar_domain, iova >> VTD_PAGE_SHIFT, &level));
 
        if (size < VTD_PAGE_SIZE << level_to_offset_bits(level))
                size = VTD_PAGE_SIZE << level_to_offset_bits(level);