diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2007-03-28 09:36:09 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-03-28 16:59:37 -0400 |
commit | bba6f6fc68e74d4572028646f61dd3505a68747e (patch) | |
tree | e593440b5944676af7ec6b85c14acbd9ac7b72c7 /arch/ia64/pci | |
parent | 4dfc896e90359df04c80da5ab08ec31e87846c43 (diff) |
[PATCH] MSI-X: fix resume crash
So I think the right solution is to simply make pci_enable_device just
flip enable bits and move the rest of the work someplace else.
However a thorough cleanup is a little extreme for this point in the
release cycle, so I think a quick hack that makes the code not stomp the
irq when msi irq's are enabled should be the first fix. Then we can
later make the code not change the irqs at all.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/ia64/pci')
-rw-r--r-- | arch/ia64/pci/pci.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/arch/ia64/pci/pci.c b/arch/ia64/pci/pci.c index 474d179966dc..f8bcccd6d417 100644 --- a/arch/ia64/pci/pci.c +++ b/arch/ia64/pci/pci.c | |||
@@ -557,14 +557,18 @@ pcibios_enable_device (struct pci_dev *dev, int mask) | |||
557 | if (ret < 0) | 557 | if (ret < 0) |
558 | return ret; | 558 | return ret; |
559 | 559 | ||
560 | return acpi_pci_irq_enable(dev); | 560 | if (!dev->msi_enabled) |
561 | return acpi_pci_irq_enable(dev); | ||
562 | return 0; | ||
561 | } | 563 | } |
562 | 564 | ||
563 | void | 565 | void |
564 | pcibios_disable_device (struct pci_dev *dev) | 566 | pcibios_disable_device (struct pci_dev *dev) |
565 | { | 567 | { |
566 | BUG_ON(atomic_read(&dev->enable_cnt)); | 568 | BUG_ON(atomic_read(&dev->enable_cnt)); |
567 | acpi_pci_irq_disable(dev); | 569 | if (!dev->msi_enabled) |
570 | acpi_pci_irq_disable(dev); | ||
571 | return 0; | ||
568 | } | 572 | } |
569 | 573 | ||
570 | void | 574 | void |