diff options
| author | Keith Busch <keith.busch@intel.com> | 2014-06-25 21:18:12 +0400 |
|---|---|---|
| committer | Jens Axboe <axboe@fb.com> | 2014-11-04 23:17:08 +0300 |
| commit | 7c1b24503873e6cfc1d31f2e55c35358fd438351 (patch) | |
| tree | 662901bd7f64313b2622dea1b8ea08c7818af94b | |
| parent | a67394790a9c52657273212647e350e36a292183 (diff) | |
| download | linux-7c1b24503873e6cfc1d31f2e55c35358fd438351.tar.xz | |
NVMe: Skip orderly shutdown on failed devices
Rather than skipping shutdown only for devices that have been removed,
skip the orderly shutdown on failed devices to avoid the long timeout
handling that inevitably happens when deleting queues on such a device.
Signed-off-by: Keith Busch <keith.busch@intel.com>
Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
| -rw-r--r-- | drivers/block/nvme-core.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/block/nvme-core.c b/drivers/block/nvme-core.c index c5f379f08f50..ac3694083e89 100644 --- a/drivers/block/nvme-core.c +++ b/drivers/block/nvme-core.c @@ -2567,11 +2567,14 @@ static void nvme_dev_list_remove(struct nvme_dev *dev) static void nvme_dev_shutdown(struct nvme_dev *dev) { int i; + u32 csts = -1; dev->initialized = 0; nvme_dev_list_remove(dev); - if (!dev->bar || (dev->bar && readl(&dev->bar->csts) == -1)) { + if (dev->bar) + csts = readl(&dev->bar->csts); + if (csts & NVME_CSTS_CFS || !(csts & NVME_CSTS_RDY)) { for (i = dev->queue_count - 1; i >= 0; i--) { struct nvme_queue *nvmeq = raw_nvmeq(dev, i); nvme_suspend_queue(nvmeq); |