diff options
author | Jens Axboe <axboe@fb.com> | 2014-11-14 09:50:19 -0700 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2014-11-20 11:10:06 -0700 |
commit | be7837e89d610046ae8dd28dc504df09261d9f91 (patch) | |
tree | 6ea5c96d1004ce9f10248312d26d70a2ed7b494f /drivers/block | |
parent | 2c30540b38d683d4c7f06d13a451f67d4362d7b1 (diff) | |
download | op-kernel-dev-be7837e89d610046ae8dd28dc504df09261d9f91.zip op-kernel-dev-be7837e89d610046ae8dd28dc504df09261d9f91.tar.gz |
NVMe: fail pci initialization if the device doesn't have any BARs
The PCI init of NVMe doesn't check for valid bars before proceeding
to map and use BAR 0. If the device is hosed (or firmware is), then
we should catch this case and give up early.
This fixes a:
[ 1662.035778] WARNING: CPU: 0 PID: 4 at arch/x86/mm/ioremap.c:63 __ioremap_check_ram+0xa7/0xc0()
and later badness on such a device.
Acked-by: Keith Busch <keith.busch@intel.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/nvme-core.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/block/nvme-core.c b/drivers/block/nvme-core.c index ba278ae..c1541652 100644 --- a/drivers/block/nvme-core.c +++ b/drivers/block/nvme-core.c @@ -2156,6 +2156,9 @@ static int nvme_dev_map(struct nvme_dev *dev) dev->entry[0].vector = pdev->irq; pci_set_master(pdev); bars = pci_select_bars(pdev, IORESOURCE_MEM); + if (!bars) + goto disable_pci; + if (pci_request_selected_regions(pdev, bars, "nvme")) goto disable_pci; |