Re: mthca double free irqs

Yinghai Lu
2009-06-13T19:28:37Z

On Fri, Jun 12, 2009 at 9:30 PM, Roland Dreier<rdreier@cisco.com> wrote:
> Something like this should help -- can you verify this fixes the double
> free of irq?
>
>  drivers/infiniband/hw/mthca/mthca_eq.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/infiniband/hw/mthca/mthca_eq.c b/drivers/infiniband/hw/mthca/mthca_eq.c
> index 28f0e0c..90e4e45 100644
> --- a/drivers/infiniband/hw/mthca/mthca_eq.c
> +++ b/drivers/infiniband/hw/mthca/mthca_eq.c
> @@ -641,9 +641,11 @@ static void mthca_free_irqs(struct mthca_dev *dev)
>        if (dev->eq_table.have_irq)
>                free_irq(dev->pdev->irq, dev);
>        for (i = 0; i < MTHCA_NUM_EQ; ++i)
> -               if (dev->eq_table.eq[i].have_irq)
> +               if (dev->eq_table.eq[i].have_irq) {
>                        free_irq(dev->eq_table.eq[i].msi_x_vector,
>                                 dev->eq_table.eq + i);
> +                       dev->eq_table.eq[i].have_irq = 0;
> +               }
>  }
>
>  static int mthca_map_reg(struct mthca_dev *dev,
>

thanks, that fixes the problem.

got

[ 688.153102] ib_mthca: Mellanox InfiniBand HCA driver v1.0 (April 4, 2008)
[ 688.164128] ib_mthca: Initializing 0000:c4:00.0
[ 688.170696] ib_mthca 0000:c4:00.0: PCI INT A -> GSI 98 (level, low) -> IRQ 98
[ 688.184033] ib_mthca 0000:c4:00.0: setting latency timer to 64
[ 688.188594] ib_mthca 0000:c4:00.0: using 64bit DMA mask
[ 688.208718] ib_mthca 0000:c4:00.0: using 64bit consistent DMA mask
[ 688.217191] reserve_memtype added 0xfbef0000-0xfbef1000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 688.232259] free_memtype request 0xfbef0000-0xfbef1000
[ 689.243254] reserve_memtype added 0xfbe80000-0xfbe81000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 689.537336] reserve_memtype added 0xfa800000-0xfa801000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 689.587286] ib_mthca 0000:c4:00.0: HCA FW version 4.7.400 is old
(4.8.200 is current).
[ 689.592067] ib_mthca 0000:c4:00.0: If you have problems, try
updating your HCA FW.
[ 689.603264] reserve_memtype added 0xfbe82000-0xfbe83000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 689.622765] alloc irq_desc for 230 on node 0
[ 689.629521] alloc kstat_irqs on node 0
[ 689.641401] ib_mthca 0000:c4:00.0: irq 230 for MSI/MSI-X
[ 689.645155] alloc irq_desc for 231 on node 0
[ 689.647072] alloc kstat_irqs on node 0
[ 689.663348] ib_mthca 0000:c4:00.0: irq 231 for MSI/MSI-X
[ 689.666000] alloc irq_desc for 232 on node 0
[ 689.682655] alloc kstat_irqs on node 0
[ 689.685768] ib_mthca 0000:c4:00.0: irq 232 for MSI/MSI-X
[ 689.701511] reserve_memtype added 0xfa802000-0xfa803000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 689.721752] reserve_memtype added 0xe5800000-0xe6000000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 689.731141] reserve_memtype added 0xe0000000-0xe4000000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 689.753919] reserve_memtype added 0xfbef0000-0xfbef1000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 689.770375] Overlap at 0xfbe80000-0xfbe81000
[ 689.782317] reserve_memtype added 0xfbe80000-0xfbe81000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 689.902837] ib_mthca 0000:c4:00.0: NOP command failed to generate
interrupt (IRQ 232).
[ 689.907900] ib_mthca 0000:c4:00.0: Trying again with MSI-X disabled.
[ 689.923292] free_memtype request 0xfbe80000-0xfbe81000
[ 689.926430] free_memtype request 0xfbef0000-0xfbef1000
[ 689.942964] free_memtype request 0xe0000000-0xe4000000
[ 689.947059] free_memtype request 0xe5800000-0xe6000000
[ 689.962869] free_memtype request 0xfa802000-0xfa803000
[ 689.967293] free_memtype request 0xfbe82000-0xfbe83000
[ 689.980961] reserve_memtype added 0xfa802000-0xfa803000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 690.003659] reserve_memtype added 0xe5800000-0xe6000000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 690.015144] reserve_memtype added 0xe0000000-0xe4000000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 690.033814] reserve_memtype added 0xfbef0000-0xfbef1000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 690.046945] Overlap at 0xfbe80000-0xfbe81000
[ 690.062803] reserve_memtype added 0xfbe80000-0xfbe81000, track
uncached-minus, req uncached-minus, ret uncached-minus
[ 690.182718] ib_mthca 0000:c4:00.0: NOP command failed to generate
interrupt (IRQ 98), aborting.
[ 690.188676] ib_mthca 0000:c4:00.0: BIOS or ACPI interrupt routing problem?
[ 690.203724] free_memtype request 0xfbe80000-0xfbe81000
[ 690.205768] free_memtype request 0xfbef0000-0xfbef1000
[ 690.224080] free_memtype request 0xe0000000-0xe4000000
[ 690.228558] free_memtype request 0xe5800000-0xe6000000
[ 690.241523] free_memtype request 0xfa802000-0xfa803000
[ 690.246196] free_memtype request 0xfbe80000-0xfbe81000
[ 690.263133] free_memtype request 0xfa800000-0xfa801000
[ 690.268628] ib_mthca 0000:c4:00.0: PCI INT A disabled
[ 690.281553] ib_mthca: probe of 0000:c4:00.0 failed with error -16

as expected.

YH