From: Martin Wilck <mwi...@suse.com>

[ Upstream commit 3f6e3246db0e6f92e784965d9d0edb8abe6c6b74 ]

Handle the special case where we have exactly one optimized path,
which we should keep using in this case.

Fixes: 75c10e732724 ("nvme-multipath: round-robin I/O policy")
Signed off-by: Martin Wilck <mwi...@suse.com>
Signed-off-by: Hannes Reinecke <h...@suse.de>
Reviewed-by: Sagi Grimberg <s...@grimberg.me>
Signed-off-by: Christoph Hellwig <h...@lst.de>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 drivers/nvme/host/multipath.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/nvme/host/multipath.c b/drivers/nvme/host/multipath.c
index 5433aa2f76017..38d25d7c6bca3 100644
--- a/drivers/nvme/host/multipath.c
+++ b/drivers/nvme/host/multipath.c
@@ -249,6 +249,12 @@ static struct nvme_ns *nvme_round_robin_path(struct 
nvme_ns_head *head,
                        fallback = ns;
        }
 
+       /* No optimized path found, re-check the current path */
+       if (!nvme_path_is_disabled(old) &&
+           old->ana_state == NVME_ANA_OPTIMIZED) {
+               found = old;
+               goto out;
+       }
        if (!fallback)
                return NULL;
        found = fallback;
-- 
2.25.1



Reply via email to