diff options
author | Jan Beulich <jbeulich@suse.com> | 2013-02-22 13:58:20 +0100 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2013-02-22 13:58:20 +0100 |
commit | 6743c50ca91da63de23ad52f037bf9eadacfb492 (patch) | |
tree | 81150bdbd5fa9d1a77fee9f2235d0802b2ea1c23 | |
parent | c705547372325f22dcc8757d63d6d8494edf53ab (diff) | |
download | xen-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.c | 3 |
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; } /* |