lkml.org 
[lkml]   [2006]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] ALSA: hda-intel - Disable MSI support by default
At Wed, 15 Nov 2006 13:32:02 -0500,
Jeff Garzik wrote:
>
> Takashi Iwai wrote:
> > The snd-hda-intel driver has a test of MSI, but it seems not working
> > on every machine. It caused non-cared interrupts and the kernel
> > disabled that irq.
>
> Possibly the test was broken. Did you have IRQF_DISABLED and
> IRQF_SHARED flags set?

I think IRQF_* is irrelevant there.
It looks like that the hardware issues INTX regardless whether MSI is
enabled or not. Thus it ends up with unexpected irqs which are never
caught by the driver (the driver expects a different irq from MSI),
and eventually the kernel kills this irq.

Possibly calling pci_intx() as you suggested might help to avoid this
situation. Can anyone test the patch below?


Takashi

---
diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index e35cfd3..bdb92b3 100644
--- a/sound/pci/hda/hda_intel.c
+++ b/sound/pci/hda/hda_intel.c
@@ -544,6 +544,7 @@ static unsigned int azx_rirb_get_respons
free_irq(chip->irq, chip);
chip->irq = -1;
pci_disable_msi(chip->pci);
+ pci_intx(chip->pci, 1);
chip->msi = 0;
if (azx_acquire_irq(chip, 1) < 0)
return -1;
@@ -830,14 +831,15 @@ static irqreturn_t azx_interrupt(int irq
{
struct azx *chip = dev_id;
struct azx_dev *azx_dev;
+ unsigned long flags;
u32 status;
int i;

- spin_lock(&chip->reg_lock);
+ spin_lock_irqsave(&chip->reg_lock, flags);

status = azx_readl(chip, INTSTS);
if (status == 0) {
- spin_unlock(&chip->reg_lock);
+ spin_unlock_irqrestore(&chip->reg_lock, flags);
return IRQ_NONE;
}

@@ -867,7 +869,7 @@ static irqreturn_t azx_interrupt(int irq
if (azx_readb(chip, STATESTS) & 0x04)
azx_writeb(chip, STATESTS, 0x04);
#endif
- spin_unlock(&chip->reg_lock);
+ spin_unlock_irqrestore(&chip->reg_lock, flags);

return IRQ_HANDLED;
}
@@ -1380,7 +1382,8 @@ static int __devinit azx_init_stream(str

static int azx_acquire_irq(struct azx *chip, int do_disconnect)
{
- if (request_irq(chip->pci->irq, azx_interrupt, IRQF_DISABLED|IRQF_SHARED,
+ if (request_irq(chip->pci->irq, azx_interrupt,
+ chip->msi ? 0 : IRQF_SHARED,
"HDA Intel", chip)) {
printk(KERN_ERR "hda-intel: unable to grab IRQ %d, "
"disabling device\n", chip->pci->irq);
@@ -1435,9 +1438,12 @@ static int azx_resume(struct pci_dev *pc
return -EIO;
}
pci_set_master(pci);
- if (chip->msi)
+ if (chip->msi) {
if (pci_enable_msi(pci) < 0)
chip->msi = 0;
+ else
+ pci_intx(pci, 0);
+ }
if (azx_acquire_irq(chip, 1) < 0)
return -EIO;
azx_init_chip(chip);
@@ -1561,9 +1567,12 @@ static int __devinit azx_create(struct s
goto errout;
}

- if (chip->msi)
+ if (chip->msi) {
if (pci_enable_msi(pci) < 0)
chip->msi = 0;
+ else
+ pci_intx(pci, 0);
+ }

if (azx_acquire_irq(chip, 0) < 0) {
err = -EBUSY;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2006-11-15 20:01    [W:0.159 / U:1.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site