Re: virt_irq_{alloc,free} inconsistency with no MSI

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Meelis Roos <mroos@xxxxxxxx>
Date: Fri, 16 Feb 2007 12:30:56 +0200 (EET)

> While investigating this (with current -git for sun4u and no 
> CONFIG_PCI_MSI)
> 
>   CC      arch/sparc64/kernel/irq.o
> cc1: warnings being treated as errors
> arch/sparc64/kernel/irq.c:113: warning: 'virt_irq_free' defined but not used
> 
> I found that virt_irq_free() is used only if CONFIG_PCI_MSI is defined 
> (for sun4v - simple to cure with a define), but virt_irq_alloc() is used 
> even without MSI enabled. Is this really correct? Do we alloc and not 
> free it in this case?

Yes, it is true :-)  We allocate all the interrupts needed for the
entire system as we probe the busses and devices.

But with MSI, the MSI vectors are allocated and liberated with
pci_enable_msi()/pci_disable_msi() and related calls.

I'll add the necessary ifdefs, thanks for the report.
-
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux