|
 |
6fdf37f |
From 05104a4e8713b27291c7bb49c1e7e68b4e243571 Mon Sep 17 00:00:00 2001
|
|
|
6fdf37f |
From: Neil Horman <nhorman@tuxdriver.com>
|
|
|
6fdf37f |
Date: Fri, 27 Sep 2013 16:53:35 +0000
|
|
|
6fdf37f |
Subject: iommu: Remove stack trace from broken irq remapping warning
|
|
|
6fdf37f |
|
|
|
6fdf37f |
The warning for the irq remapping broken check in intel_irq_remapping.c is
|
|
|
6fdf37f |
pretty pointless. We need the warning, but we know where its comming from, the
|
|
|
6fdf37f |
stack trace will always be the same, and it needlessly triggers things like
|
|
|
6fdf37f |
Abrt. This changes the warning to just print a text warning about BIOS being
|
|
|
6fdf37f |
broken, without the stack trace, then sets the appropriate taint bit. Since we
|
|
|
6fdf37f |
automatically disable irq remapping, theres no need to contiue making Abrt jump
|
|
|
6fdf37f |
at this problem
|
|
|
6fdf37f |
|
|
|
6fdf37f |
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
|
|
|
6fdf37f |
CC: Joerg Roedel <joro@8bytes.org>
|
|
|
6fdf37f |
CC: Bjorn Helgaas <bhelgaas@google.com>
|
|
|
6fdf37f |
CC: Andy Lutomirski <luto@amacapital.net>
|
|
|
6fdf37f |
CC: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
|
|
|
6fdf37f |
CC: Sebastian Andrzej Siewior <sebastian@breakpoint.cc>
|
|
|
6fdf37f |
Signed-off-by: Joerg Roedel <joro@8bytes.org>
|
|
|
6fdf37f |
---
|
|
|
6fdf37f |
diff --git a/drivers/iommu/intel_irq_remapping.c b/drivers/iommu/intel_irq_remapping.c
|
|
|
6fdf37f |
index f71673d..b97d70b 100644
|
|
|
6fdf37f |
--- a/drivers/iommu/intel_irq_remapping.c
|
|
|
6fdf37f |
+++ b/drivers/iommu/intel_irq_remapping.c
|
|
|
6fdf37f |
@@ -525,12 +525,13 @@ static int __init intel_irq_remapping_supported(void)
|
|
|
6fdf37f |
if (disable_irq_remap)
|
|
|
6fdf37f |
return 0;
|
|
|
6fdf37f |
if (irq_remap_broken) {
|
|
|
6fdf37f |
- WARN_TAINT(1, TAINT_FIRMWARE_WORKAROUND,
|
|
|
6fdf37f |
- "This system BIOS has enabled interrupt remapping\n"
|
|
|
6fdf37f |
- "on a chipset that contains an erratum making that\n"
|
|
|
6fdf37f |
- "feature unstable. To maintain system stability\n"
|
|
|
6fdf37f |
- "interrupt remapping is being disabled. Please\n"
|
|
|
6fdf37f |
- "contact your BIOS vendor for an update\n");
|
|
|
6fdf37f |
+ printk(KERN_WARNING
|
|
|
6fdf37f |
+ "This system BIOS has enabled interrupt remapping\n"
|
|
|
6fdf37f |
+ "on a chipset that contains an erratum making that\n"
|
|
|
6fdf37f |
+ "feature unstable. To maintain system stability\n"
|
|
|
6fdf37f |
+ "interrupt remapping is being disabled. Please\n"
|
|
|
6fdf37f |
+ "contact your BIOS vendor for an update\n");
|
|
|
6fdf37f |
+ add_taint(TAINT_FIRMWARE_WORKAROUND, LOCKDEP_STILL_OK);
|
|
|
6fdf37f |
disable_irq_remap = 1;
|
|
|
6fdf37f |
return 0;
|
|
|
6fdf37f |
}
|
|
|
6fdf37f |
--
|
|
|
6fdf37f |
cgit v0.9.2
|