On Tue, Jun 02, 2015 at 01:44:58PM +0200, Takashi Iwai wrote:
At Tue, 2 Jun 2015 12:10:34 +0100, Mark Brown wrote:
On Tue, Jun 02, 2015 at 12:48:50PM +0200, Takashi Iwai wrote:
Mark Brown wrote:
Why does the driver care though? IRQF_SHARED is advertising the capabilities of the hander, not a requirement on the hardware - if the interrupt physically can't be shared then the ability to share it will never get used but that shouldn't matter.
Because the kernel doesn't guarantee the exclusiveness of irq handler registration as long as you pass IRQF_SHARED. That is, if we keep IRQF_SHARED and another driver tries to request_irq() for the same irq with again IRQF_SHARED. But this shouldn't happen with MSI.
Sure, but how could that happen (given that the interrupt physically can't be shared) and surely individual client drivers are the wrong place to do this -
Oh how can you trust BIOS setup? :) A wrong numbered IRQ is a most frequently seen problem (mostly not about MSI, though).
well for SKL the systems are supposed to be not using MSI(my orignal thought removing msi part and going to shared irq only), but we know how easily these things get messed up, so having this sounds better here