aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Beulich <jbeulich@suse.com>2013-02-22 13:58:20 +0100
committerJan Beulich <jbeulich@suse.com>2013-02-22 13:58:20 +0100
commit6743c50ca91da63de23ad52f037bf9eadacfb492 (patch)
tree81150bdbd5fa9d1a77fee9f2235d0802b2ea1c23
parentc705547372325f22dcc8757d63d6d8494edf53ab (diff)
downloadxen-6743c50ca91da63de23ad52f037bf9eadacfb492.tar.gz
xen-6743c50ca91da63de23ad52f037bf9eadacfb492.tar.bz2
xen-6743c50ca91da63de23ad52f037bf9eadacfb492.zip
AMD IOMMU: don't BUG() when we don't have to
find_iommu_for_device() can easily return NULL instead, as all of its callers are prepared for that. Signed-off-by: Jan Beulich <jbeulich@suse.com> master changeset: f547d42ec0306cdceffb8f7603c7e6f8977cf398 master date: 2013-02-18 09:37:35 +0100
-rw-r--r--xen/drivers/passthrough/amd/pci_amd_iommu.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/xen/drivers/passthrough/amd/pci_amd_iommu.c b/xen/drivers/passthrough/amd/pci_amd_iommu.c
index ccb39eae3b..21b7460e02 100644
--- a/xen/drivers/passthrough/amd/pci_amd_iommu.c
+++ b/xen/drivers/passthrough/amd/pci_amd_iommu.c
@@ -30,8 +30,7 @@ extern struct ivrs_mappings *ivrs_mappings;
struct amd_iommu *find_iommu_for_device(int bdf)
{
- BUG_ON ( bdf >= ivrs_bdf_entries );
- return ivrs_mappings[bdf].iommu;
+ return bdf < ivrs_bdf_entries ? ivrs_mappings[bdf].iommu : NULL;
}
/*