PCI MSI: Return if alloc_msi_entry for MSI-X failed
authorHidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
Wed, 24 Jun 2009 03:08:27 +0000 (12:08 +0900)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Mon, 29 Jun 2009 19:10:10 +0000 (12:10 -0700)
commit0d07348931daef854aca8c834a89f1a99ba4ff2b
tree3cc936fd7f31d00ce9c452ac39ce772f628fccc3
parent2bfdd79eaa0043346e773ba5f6cfd811ea31b73d
PCI MSI: Return if alloc_msi_entry for MSI-X failed

In current code it continues setup even if alloc_msi_entry() for MSI-X
is failed due to lack of memory.  It means arch_setup_msi_irqs() might
be called with msi_desc entries less than its argument nvec.

At least x86's arch_setup_msi_irqs() uses list_for_each_entry() for
dev->msi_list that suspected to have entries same numbers as nvec, and
it doesn't check the number of allocated vectors and passed arg nvec.
Therefore it will result in success of pci_enable_msix(), with less
vectors allocated than requested.

This patch fixes the error route to return -ENOMEM, instead of continuing
the setup (proposed by Matthew Wilcox).

Note that there is no iounmap in msi_free_irqs() if no msi_disc is
allocated.

Reviewed-by: Matthew Wilcox <matthew@wil.cx>
Signed-off-by: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/msi.c