diff options
| author |   Joerg Roedel <jroedel@suse.de> | 2018-05-03 15:25:17 +0200 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2018-05-03 16:32:11 +0200 |
| commit | a85894cd779455fab0086cfcb5c9f65c3706e1c6 (patch) | |
| tree | 3d4c642f27a35f59c6bef8ca14a2e52cd1064db7 /drivers/iommu | |
| parent | iommu/vt-d: fix shift-out-of-bounds in bug checking (diff) | |
| download | linux-dev-a85894cd779455fab0086cfcb5c9f65c3706e1c6.tar.xz linux-dev-a85894cd779455fab0086cfcb5c9f65c3706e1c6.zip | |
iommu/vt-d: Use WARN_ON_ONCE instead of BUG_ON in qi_flush_dev_iotlb()
A misaligned address is only worth a warning, and not
stopping the while execution path with a BUG_ON().
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu')
| -rw-r--r-- | drivers/iommu/dmar.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c index e4ae6003f470..460bed4fc5b1 100644 --- a/drivers/iommu/dmar.c +++ b/drivers/iommu/dmar.c @@ -1345,7 +1345,7 @@ void qi_flush_dev_iotlb(struct intel_iommu *iommu, u16 sid, u16 qdep, struct qi_desc desc; if (mask) { - BUG_ON(addr & ((1ULL << (VTD_PAGE_SHIFT + mask)) - 1)); + WARN_ON_ONCE(addr & ((1ULL << (VTD_PAGE_SHIFT + mask)) - 1)); addr |= (1ULL << (VTD_PAGE_SHIFT + mask - 1)) - 1; desc.high = QI_DEV_IOTLB_ADDR(addr) | QI_DEV_IOTLB_SIZE; } else |