Pārlūkot izejas kodu

NVMe: Merge issue on character device bring-up

A recent patch made it possible to bring up the character handle when the
device is responsive but not accepting a set-features command. Another
recent patch moved the initialization that requires we move where the
checks for this condition occur. This patch merges these two ideas so
it works much as before.

Signed-off-by: Keith Busch <keith.busch@intel.com>
Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
Keith Busch 12 gadi atpakaļ
vecāks
revīzija
d82e8bfdef
1 mainītis faili ar 8 papildinājumiem un 4 dzēšanām
  1. 8 4
      drivers/block/nvme-core.c

+ 8 - 4
drivers/block/nvme-core.c

@@ -2135,10 +2135,10 @@ static int nvme_dev_start(struct nvme_dev *dev)
 	spin_unlock(&dev_list_lock);
 	spin_unlock(&dev_list_lock);
 
 
 	result = nvme_setup_io_queues(dev);
 	result = nvme_setup_io_queues(dev);
-	if (result)
+	if (result && result != -EBUSY)
 		goto disable;
 		goto disable;
 
 
-	return 0;
+	return result;
 
 
  disable:
  disable:
 	spin_lock(&dev_list_lock);
 	spin_lock(&dev_list_lock);
@@ -2177,13 +2177,17 @@ static int nvme_probe(struct pci_dev *pdev, const struct pci_device_id *id)
 		goto release;
 		goto release;
 
 
 	result = nvme_dev_start(dev);
 	result = nvme_dev_start(dev);
-	if (result)
+	if (result) {
+		if (result == -EBUSY)
+			goto create_cdev;
 		goto release_pools;
 		goto release_pools;
+	}
 
 
 	result = nvme_dev_add(dev);
 	result = nvme_dev_add(dev);
-	if (result && result != -EBUSY)
+	if (result)
 		goto shutdown;
 		goto shutdown;
 
 
+ create_cdev:
 	scnprintf(dev->name, sizeof(dev->name), "nvme%d", dev->instance);
 	scnprintf(dev->name, sizeof(dev->name), "nvme%d", dev->instance);
 	dev->miscdev.minor = MISC_DYNAMIC_MINOR;
 	dev->miscdev.minor = MISC_DYNAMIC_MINOR;
 	dev->miscdev.parent = &pdev->dev;
 	dev->miscdev.parent = &pdev->dev;