From: Christophe JAILLET christophe.jaillet@wanadoo.fr
[ Upstream commit 745610c4a3e3baaebf6d1f8cd5b4d82892432520 ]
If 'intel_cap_audit()' fails, we should return directly, as already done in the surrounding error handling path.
Fixes: ad3d19029979 ("iommu/vt-d: Audit IOMMU Capabilities and add helper functions") Signed-off-by: Christophe JAILLET christophe.jaillet@wanadoo.fr Acked-by: Lu Baolu baolu.lu@linux.intel.com Link: https://lore.kernel.org/r/98d531caabe66012b4fffc7813fd4b9470afd517.161812477... Signed-off-by: Joerg Roedel jroedel@suse.de Signed-off-by: Sasha Levin sashal@kernel.org --- drivers/iommu/intel/irq_remapping.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/iommu/intel/irq_remapping.c b/drivers/iommu/intel/irq_remapping.c index 611ef5243cb6..5c16ebe037a1 100644 --- a/drivers/iommu/intel/irq_remapping.c +++ b/drivers/iommu/intel/irq_remapping.c @@ -736,7 +736,7 @@ static int __init intel_prepare_irq_remapping(void) return -ENODEV;
if (intel_cap_audit(CAP_AUDIT_STATIC_IRQR, NULL)) - goto error; + return -ENODEV;
if (!dmar_ir_support()) return -ENODEV;