From: Christoph Hellwig Date: Tue, 26 Apr 2016 11:51:58 +0000 (+0200) Subject: nvme: tighten up state check for namespace scanning X-Git-Tag: v4.7-rc1~156^2~35 X-Git-Url: https://asedeno.scripts.mit.edu/gitweb/?a=commitdiff_plain;h=92911a55d42084cd285250c275d9f238783638c2;p=linux.git nvme: tighten up state check for namespace scanning We only should be scanning namespaces if the controller is live. Currently we call the function just before setting it live, so fix the code up to move the call to nvme_queue_scan to just below the state change. Signed-off-by: Christoph Hellwig Reviewed-by: Sagi Grimberg Acked-by Jon Derrick: Signed-off-by: Jens Axboe --- diff --git a/drivers/nvme/host/pci.c b/drivers/nvme/host/pci.c index 0cee23651aa7..9b2deba0bc91 100644 --- a/drivers/nvme/host/pci.c +++ b/drivers/nvme/host/pci.c @@ -272,7 +272,7 @@ static void nvme_queue_scan(struct nvme_dev *dev) * Do not queue new scan work when a controller is reset during * removal. */ - if (dev->ctrl.state != NVME_CTRL_DELETING) + if (dev->ctrl.state == NVME_CTRL_LIVE) queue_work(nvme_workq, &dev->scan_work); } @@ -1659,7 +1659,6 @@ static int nvme_dev_add(struct nvme_dev *dev) nvme_free_queues(dev, dev->online_queues); } - nvme_queue_scan(dev); return 0; } @@ -1893,6 +1892,9 @@ static void nvme_reset_work(struct work_struct *work) dev_warn(dev->ctrl.device, "failed to mark controller live\n"); goto out; } + + if (dev->online_queues > 1) + nvme_queue_scan(dev); return; out: