Commit 16d279d2 authored by Andi Kleen's avatar Andi Kleen Committed by Andi Kleen

[PATCH] x86: Fix verify_quirk_intel_irqbalance()

Fix verify_quirk_intel_irqbalance(). genapic checks should really
happen only on affected versions of the E7520/E7320/E7525 based platforms.

AK: This should akpm's Coyote SDV
Signed-off-by: default avatarSuresh Siddha <suresh.b.siddha@intel.com>
Signed-off-by: default avatarAndi Kleen <ak@suse.de>
parent e0f27981
...@@ -10,6 +10,26 @@ ...@@ -10,6 +10,26 @@
#if defined(CONFIG_X86_IO_APIC) && defined(CONFIG_SMP) && defined(CONFIG_PCI) #if defined(CONFIG_X86_IO_APIC) && defined(CONFIG_SMP) && defined(CONFIG_PCI)
static void __devinit verify_quirk_intel_irqbalance(struct pci_dev *dev) static void __devinit verify_quirk_intel_irqbalance(struct pci_dev *dev)
{ {
u8 config, rev;
u32 word;
/* BIOS may enable hardware IRQ balancing for
* E7520/E7320/E7525(revision ID 0x9 and below)
* based platforms.
* For those platforms, make sure that the genapic is set to 'flat'
*/
pci_read_config_byte(dev, PCI_CLASS_REVISION, &rev);
if (rev > 0x9)
return;
/* enable access to config space*/
pci_read_config_byte(dev, 0xf4, &config);
pci_write_config_byte(dev, 0xf4, config|0x2);
/* read xTPR register */
raw_pci_ops->read(0, 0, 0x40, 0x4c, 2, &word);
if (!(word & (1 << 13))) {
#ifdef CONFIG_X86_64 #ifdef CONFIG_X86_64
if (genapic != &apic_flat) if (genapic != &apic_flat)
panic("APIC mode must be flat on this system\n"); panic("APIC mode must be flat on this system\n");
...@@ -17,6 +37,11 @@ static void __devinit verify_quirk_intel_irqbalance(struct pci_dev *dev) ...@@ -17,6 +37,11 @@ static void __devinit verify_quirk_intel_irqbalance(struct pci_dev *dev)
if (genapic != &apic_default) if (genapic != &apic_default)
panic("APIC mode must be default(flat) on this system. Use apic=default\n"); panic("APIC mode must be default(flat) on this system. Use apic=default\n");
#endif #endif
}
/* put back the original value for config space*/
if (!(config & 0x2))
pci_write_config_byte(dev, 0xf4, config);
} }
void __init quirk_intel_irqbalance(void) void __init quirk_intel_irqbalance(void)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment