[lkml]   [2010]   [Jan]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectRe: [BUG] tip tree - No irq handler for vector (irq -1)
    On Thu, 2010-01-28 at 19:14 -0800, Jin Dongming wrote:
    > Hi,
    > Maybe this problem is caused by that FIRST_DEVICE_VECTOR(IRQ 0x30)
    > does not assigned really.

    No. The problem is that this system's logical flat is behaving weird.

    Even when the IO-APIC RTE says send the interrupt to cpu-0, it sends the
    interrupt to cpu-1 (for which we haven't really initialized the vector
    to irq mapping).

    Actually this case is documented by Eric Biederman in the code like

    static void flat_vector_allocation_domain(int cpu, struct cpumask
    /* Careful. Some cpus do not strictly honor the set of cpus
    * specified in the interrupt destination when using lowest
    * priority interrupt delivery mode.
    * In particular there was a hyperthreading cpu observed to
    * deliver interrupts to the wrong hyperthread when only one
    * hyperthread was specified in the interrupt desitination.
    cpumask_bits(retmask)[0] = APIC_ALL_CPUS;

    And the previous kernel codes were handling this and my recent changes
    to vector assignments broke this behavior. My previous fix addressed a
    portion of this issue and hence Zefan's keyboard etc started working.
    Nevertheless there is one more corner case which needs to get addressed
    and if my understanding is correct, the appended patch (ontop of the
    patch that Zefan has already tested) should fix the issue.

    Zefan, can you also apply this fix (untested/uncompiled) on top of the
    previous fix and see if it resolves the issue? If so, I will add
    appropriate changelog, comments and send it tomorrow morning for -tip
    consideration after more testing. Thanks.

    diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c
    index 678d0b8..5d9896b 100644
    --- a/arch/x86/kernel/smpboot.c
    +++ b/arch/x86/kernel/smpboot.c
    @@ -241,6 +241,13 @@ static void __cpuinit smp_callin(void)

    + /*
    + * Need to setup vector mappings before we enable interrupts.
    + */
    + lock_vector_lock();
    + __setup_vector_irq(smp_processor_id());
    + unlock_vector_lock();
    * Get our bogomips.
    @@ -315,7 +322,6 @@ notrace static void __cpuinit start_secondary(void
    - __setup_vector_irq(smp_processor_id());
    set_cpu_online(smp_processor_id(), true);

     \ /
      Last update: 2010-01-29 07:55    [W:0.025 / U:18.284 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site