[PATCH] linux_2.4.22-pre2_clear-smi-fix_A0

john stultz (johnstul@us.ibm.com)
25 Jun 2003 14:05:14 -0700


Marcelo, all,

This is a backported fix from 2.5. Some of our more recent hardware
requires that SMIs are routed through the IOAPIC, thus when we
clear_IO_APIC() at boot time, we clear the BIOS initialized SMI pin.
This basically clobbers the SMI so we can then never make the transition
into ACPI mode. Additionally various other SMI functions cease to work.
This patch simply reads the apic entry in clear_IO_APIC to make sure the
delivery_mode isn't dest_SMI. If it is, we leave the apic entry alone
and return.

With this patch, the box boots and SMIs function properly.

Please consider for inclusion.

thanks
-john

diff -Nru a/arch/i386/kernel/io_apic.c b/arch/i386/kernel/io_apic.c
--- a/arch/i386/kernel/io_apic.c Wed Jun 25 13:27:07 2003
+++ b/arch/i386/kernel/io_apic.c Wed Jun 25 13:27:07 2003
@@ -169,6 +169,14 @@
{
struct IO_APIC_route_entry entry;
unsigned long flags;
+
+ /* Check delivery_mode to be sure we're not clearing an SMI pin */
+ spin_lock_irqsave(&ioapic_lock, flags);
+ *(((int*)&entry) + 0) = io_apic_read(apic, 0x10 + 2 * pin);
+ *(((int*)&entry) + 1) = io_apic_read(apic, 0x11 + 2 * pin);
+ spin_unlock_irqrestore(&ioapic_lock, flags);
+ if (entry.delivery_mode == dest_SMI)
+ return;

/*
* Disable it in the IO-APIC irq-routing table:

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/