PCI: Fail MSI-X mappings if there's no space assigned to MSI-X BAR
authorYijing Wang <wangyijing@huawei.com>
Wed, 28 Jan 2015 01:52:17 +0000 (09:52 +0800)
committerBjorn Helgaas <bhelgaas@google.com>
Wed, 28 Jan 2015 15:25:57 +0000 (09:25 -0600)
commit6a878e5085fe97bd1e222b7883a1b815fcbbe4ed
tree2ffcc34c8799f2c387c46c3441aa8c1072000e03
parent97bf6af1f928216fd6c5a66e8a57bfa95a659672
PCI: Fail MSI-X mappings if there's no space assigned to MSI-X BAR

Unlike MSI, which is configured via registers in the MSI capability in
Configuration Space, MSI-X is configured via tables in Memory Space.
These MSI-X tables are mapped by a device BAR, and if no Memory Space
has been assigned to the BAR, MSI-X cannot be used.

Fail MSI-X setup if no space has been assigned for the BAR.

Previously, we ioremapped the MSI-X table even if the resource hadn't been
assigned.  In this case, the resource address is undefined (and is often
zero), which may lead to warnings or oopses in this path:

  pci_enable_msix
    msix_capability_init
      msix_map_region
        ioremap_nocache

The PCI core sets resource flags to zero when it can't assign space for the
resource (see reset_resource()).  There are also some cases where it sets
the IORESOURCE_UNSET flag, e.g., pci_reassigndev_resource_alignment(),
pci_assign_resource(), etc.  So we must check for both cases.

[bhelgaas: changelog]
Reported-by: Zhang Jukuo <zhangjukuo@huawei.com>
Tested-by: Zhang Jukuo <zhangjukuo@huawei.com>
Signed-off-by: Yijing Wang <wangyijing@huawei.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
arch/x86/pci/xen.c
drivers/pci/msi.c