nvme: delete NVME_CTRL_LIVE --> NVME_CTRL_CONNECTING transition

There is no logical reason to move from live state to connecting
state. In case of initial connection establishment, the transition
should be NVME_CTRL_NEW --> NVME_CTRL_CONNECTING --> NVME_CTRL_LIVE.
In case of error recovery or reset, the transition should be
NVME_CTRL_LIVE --> NVME_CTRL_RESETTING --> NVME_CTRL_CONNECTING -->
NVME_CTRL_LIVE.

Signed-off-by: Max Gurtovoy <maxg@mellanox.com>
Reviewed-by: James Smart <james.smart@broadcom.com>
Signed-off-by: Sagi Grimberg <sagi@grimberg.me>

diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
index 86dca2919e1911642b15ece5763e54b9e6d6ffb1..1f9278364196e90d391fb98cff07c6340790ab9e 100644 (file)
--- a/drivers/nvme/host/core.c
+++ b/drivers/nvme/host/core.c
@@ -297,7 +297,6 @@ bool nvme_change_ctrl_state(struct nvme_ctrl *ctrl,
        case NVME_CTRL_CONNECTING:
                switch (old_state) {
                case NVME_CTRL_NEW:
-               case NVME_CTRL_LIVE:
                case NVME_CTRL_RESETTING:
                        changed = true;
                        /* FALLTHRU */