Messages in this thread | | | Date | Sat, 2 Mar 2013 16:59:42 +0100 | From | Andreas Mohr <> | Subject | Re: [PATCH] irq: add quirk for broken interrupt remapping on 55XX chipsets |
| |
Hi,
> if ((revision == 0x13) && irq_remapping_enabled) { > + pr_warn("WARNING WARNING WARNING WARNING WARNING > WARNING\n" > + "This system BIOS has enabled interrupt > remapping\n" > + "on a chipset that contains an errata making > that\n" > + "feature unstable. Please reboot with > nointremap\n" > + "added to the kernel command line and contact\n" > + "your BIOS vendor for an update"); > + }
Forgive me, but ISTR that there's a special BIOS firmware quirk bug annotating logger warning message mechanism (have I managed to hit all keywords yet? ;) in the kernel which might be useful in this case.
OK, found something (but I don't think it was the mechanism that ISTR - perhaps it got modernized?):
include/linux/printk.h:
/* * FW_BUG * Add this to a message where you are sure the firmware is buggy or * behaves * really stupid or out of spec. Be aware that the responsible BIOS * developer * should be able to fix this issue or at least get a concrete idea of * the * problem by reading your message without the need of looking at the * kernel * code. * * Use it for definite and high priority BIOS bugs. * * FW_WARN * Use it for not that clear (e.g. could the kernel messed up things * already?) * and medium priority BIOS bugs. * * FW_INFO * Use this one if you want to tell the user or vendor about something * suspicious, but generally harmless related to the firmware. * * Use it for information or very low priority BIOS bugs. */
So perhaps it is appropriate to be used here.
HTH,
Andreas Mohr
| |