nvme: delete NVME_CTRL_LIVE --> NVME_CTRL_CONNECTING transition

"Linux Kernel Mailing List" <[email protected]> Sat, 17 Feb 2018 19:17:15 +0000 (UTC)
Newsgroups gmane.linux.kernel.commits.head
Message-ID <[email protected]>
Web:        https://git.kernel.org/torvalds/c/3096a739d2ccbfd6a626e388228a16558f76d79d
Commit:     3096a739d2ccbfd6a626e388228a16558f76d79d
Parent:     b754a32c66772e6510acd92237aadd4cf227ae39
Refname:    refs/heads/master
Author:     Max Gurtovoy <[email protected]>
AuthorDate: Wed Jan 31 18:31:26 2018 +0200
Committer:  Sagi Grimberg <[email protected]>
CommitDate: Thu Feb 8 18:35:54 2018 +0200

    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 <[email protected]>
    Reviewed-by: James Smart <[email protected]>
    Signed-off-by: Sagi Grimberg <[email protected]>
---
 drivers/nvme/host/core.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
index 86dca2919e19..1f9278364196 100644
--- 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 */
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html