Re: 2.6.35-rc* regression: ERROR: Unable to locate IOAPIC for GSI *

[Yinghai Lu]

On Thu, Jul 8, 2010 at 1:27 PM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote:
> On Thu, Jul 8, 2010 at 6:53 AM, Meelis Roos <mroos@xxxxxxxx> wrote:
>> Just tried 2.6.35-rc4-00086-g2aa72f6 on Celeron 900 + i815 chipset PC.
>> I get these errors during bootup, and all interrupts become XT-PIC:
>>
>> [    0.087302] pnp: PnP ACPI init
>> [    0.087482] ACPI: bus type pnp registered
>> [    0.087999] ERROR: Unable to locate IOAPIC for GSI 8
>> [    0.087999] ERROR: Unable to locate IOAPIC for GSI 13
>> [    0.087999] ERROR: Unable to locate IOAPIC for GSI 12
>> [    0.088091] ERROR: Unable to locate IOAPIC for GSI 1
>> [    0.089013] ERROR: Unable to locate IOAPIC for GSI 6
>> [    0.090612] ERROR: Unable to locate IOAPIC for GSI 4
>> [    0.092235] ERROR: Unable to locate IOAPIC for GSI 3
>> [    0.094382] ERROR: Unable to locate IOAPIC for GSI 7
>> [    0.097470] pnp: PnP ACPI: found 13 devices
>> [    0.097625] ACPI: ACPI bus type pnp unregistered
>>
>> 2.6.35-rc2-00001-g386f40c behaves the same.
>>
>> 2.6.34 worked fine, it reports just
>> [    0.104513] pnp: PnP ACPI init
>> [    0.104683] ACPI: bus type pnp registered
>> [    0.116002] pnp: PnP ACPI: found 13 devices
>> [    0.116158] ACPI: ACPI bus type pnp unregistered
>
> Do you have bootlog for 2.6.34 with "debug apic=debug"?
>
>>
>> Will bisect but it takes some time.
>>
>> Full dmesg:
>> [    0.000000] ACPI: RSDP 000ff980 00014 (v00 AMI   )
>> [    0.000000] ACPI: RSDT 1fff0000 0002C (v01 D815EA D815EEA2 20021106 MSFT 00001011)
>> [    0.000000] ACPI: FACP 1fff1000 00074 (v01 D815EA EA81510A 20021106 MSFT 00001011)
>> [    0.000000] ACPI: DSDT 1ffe0000 030E4 (v01 D815E2 EA81520A 00000023 MSFT 0100000B)
>> [    0.000000] ACPI: FACS 1fff8000 00040
>> [    0.000000] ACPI: SSDT 1ffe30e4 00035 (v01 D815EA EA81510A 00000015 MSFT 0100000B)
>
> no MADT
>
>> [    0.000000] ACPI: PM-Timer IO Port: 0x408
>> [    0.000000] Found and enabled local APIC!
>> [    0.000000] nr_irqs_gsi: 16
>
>> [    0.021087] Enabling APIC mode:  Flat.  Using 0 I/O APICs
> no io apic.

related commit:

commit 9a0a91bb56d2915cdb8585717de38376ad20fef9
Author: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
Date:   Tue Mar 30 01:07:03 2010 -0700

    x86, acpi/irq: Teach acpi_get_override_irq to take a gsi not an isa_irq

    In perverse acpi implementations the isa irqs are not identity mapped
    to the first 16 gsi.  Furthermore at least the extended interrupt
    resource capability may return gsi's and not isa irqs.  So since
    what we get from acpi is a gsi teach acpi_get_overrride_irq to
    operate on a gsi instead of an isa_irq.

    Signed-off-by: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
    LKML-Reference: <1269936436-7039-2-git-send-email-ebiederm@xxxxxxxxxxxx>
    Signed-off-by: H. Peter Anvin <hpa@xxxxxxxxx>

diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c
index 127b871..73ec928 100644
--- a/arch/x86/kernel/apic/io_apic.c
+++ b/arch/x86/kernel/apic/io_apic.c
@@ -4082,22 +4082,27 @@ int __init io_apic_get_version(int ioapic)
        return reg_01.bits.version;
 }

-int acpi_get_override_irq(int bus_irq, int *trigger, int *polarity)
+int acpi_get_override_irq(u32 gsi, int *trigger, int *polarity)
 {
-       int i;
+       int ioapic, pin, idx;

        if (skip_ioapic_setup)
                return -1;

-       for (i = 0; i < mp_irq_entries; i++)
-               if (mp_irqs[i].irqtype == mp_INT &&
-                   mp_irqs[i].srcbusirq == bus_irq)
-                       break;
-       if (i >= mp_irq_entries)
+       ioapic = mp_find_ioapic(gsi);
+       if (ioapic < 0)
+               return -1;
+
+       pin = mp_find_ioapic_pin(ioapic, gsi);
+       if (pin < 0)
+               return -1;
+
+       idx = find_irq_entry(ioapic, pin, mp_INT);
+       if (idx < 0)
                return -1;

-       *trigger = irq_trigger(i);
-       *polarity = irq_polarity(i);
+       *trigger = irq_trigger(idx);
+       *polarity = irq_polarity(idx);
        return 0;
 }


Meelis' system doesn't have madt and mptable.

and eric patch change to call mp_find_ioapic, and mp_find_ioapic_pin.

so could just check

if (nr_ioapics < 1)
     return -1;

in acpi_get_override_irq()

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