lkml.org 
[lkml]   [2010]   [Mar]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 05/15] x86 acpi/irq: Generalize mp_config_acpi_legacy_irqs
    Date
    From: Eric W. Biederman <ebiederm@xmission.com>

    Remove the assumption that there is not an override for isa irq 0.
    Instead lookup the gsi and from that lookup the ioapic and pin of each
    isa irq indivdually.

    In general this should not have any behavioural affect but in
    perverse cases this gets all of the details correct, instead of
    doing something weird.

    Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
    ---
    arch/x86/kernel/acpi/boot.c | 30 ++++++++++++++++++------------
    1 files changed, 18 insertions(+), 12 deletions(-)

    diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c
    index 1742888..9c48e99 100644
    --- a/arch/x86/kernel/acpi/boot.c
    +++ b/arch/x86/kernel/acpi/boot.c
    @@ -960,8 +960,6 @@ void __init mp_override_legacy_irq(u8 bus_irq, u8 polarity, u8 trigger, u32 gsi)
    void __init mp_config_acpi_legacy_irqs(void)
    {
    int i;
    - int ioapic;
    - unsigned int dstapic;
    struct mpc_intsrc mp_irq;

    #if defined (CONFIG_MCA) || defined (CONFIG_EISA)
    @@ -982,19 +980,27 @@ void __init mp_config_acpi_legacy_irqs(void)
    #endif

    /*
    - * Locate the IOAPIC that manages the ISA IRQs (0-15).
    - */
    - ioapic = mp_find_ioapic(0);
    - if (ioapic < 0)
    - return;
    - dstapic = mp_ioapics[ioapic].apicid;
    -
    - /*
    * Use the default configuration for the IRQs 0-15. Unless
    * overridden by (MADT) interrupt source override entries.
    */
    for (i = 0; i < 16; i++) {
    + int ioapic, pin;
    + unsigned int dstapic;
    int idx;
    + u32 gsi;
    +
    + /* Locate the gsi that irq i maps to. */
    + if (acpi_isa_irq_to_gsi(i, &gsi))
    + continue;
    +
    + /*
    + * Locate the IOAPIC that manages the ISA IRQ.
    + */
    + ioapic = mp_find_ioapic(gsi);
    + if (ioapic < 0)
    + continue;
    + pin = mp_find_ioapic_pin(ioapic, gsi);
    + dstapic = mp_ioapics[ioapic].apicid;

    for (idx = 0; idx < mp_irq_entries; idx++) {
    struct mpc_intsrc *irq = mp_irqs + idx;
    @@ -1004,7 +1010,7 @@ void __init mp_config_acpi_legacy_irqs(void)
    break;

    /* Do we already have a mapping for this IOAPIC pin */
    - if (irq->dstapic == dstapic && irq->dstirq == i)
    + if (irq->dstapic == dstapic && irq->dstirq == pin)
    break;
    }

    @@ -1019,7 +1025,7 @@ void __init mp_config_acpi_legacy_irqs(void)
    mp_irq.dstapic = dstapic;
    mp_irq.irqtype = mp_INT;
    mp_irq.srcbusirq = i; /* Identity mapped */
    - mp_irq.dstirq = i;
    + mp_irq.dstirq = pin;

    save_mp_irq(&mp_irq);
    }
    --
    1.6.5.2.143.g8cc62


    \
     
     \ /
      Last update: 2010-03-30 10:09    [W:0.023 / U:0.292 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site