diff options
| author |   Daniel Wagner <wagi@kernel.org> | 2025-01-09 14:30:47 +0100 |
|---|---|---|
| committer |   Keith Busch <kbusch@kernel.org> | 2025-01-23 10:04:25 -0800 |
| commit | d3d380eded7ee5fc2fc53b3b0e72365ded025c4a (patch) | |
| tree | c0191842744c27282f9f55d49d66c8535b2929ec | |
| parent | nvme-pci: Add TUXEDO IBP Gen9 to Samsung sleep quirk (diff) | |
| download | wireguard-linux-d3d380eded7ee5fc2fc53b3b0e72365ded025c4a.tar.xz wireguard-linux-d3d380eded7ee5fc2fc53b3b0e72365ded025c4a.zip | |
nvme-fc: go straight to connecting state when initializing
The initial controller initialization mimiks the reconnect loop
behavior by switching from NEW to RESETTING and then to CONNECTING.
The transition from NEW to CONNECTING is a valid transition, so there is
no point entering the RESETTING state. TCP and RDMA also transition
directly to CONNECTING state.
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Daniel Wagner <wagi@kernel.org>
Signed-off-by: Keith Busch <kbusch@kernel.org>
Diffstat (limited to '')
| -rw-r--r-- | drivers/nvme/host/fc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/nvme/host/fc.c b/drivers/nvme/host/fc.c index 094be164ffdc..7409da42b9ee 100644 --- a/drivers/nvme/host/fc.c +++ b/drivers/nvme/host/fc.c @@ -3578,8 +3578,7 @@ nvme_fc_init_ctrl(struct device *dev, struct nvmf_ctrl_options *opts, list_add_tail(&ctrl->ctrl_list, &rport->ctrl_list); spin_unlock_irqrestore(&rport->lock, flags); - if (!nvme_change_ctrl_state(&ctrl->ctrl, NVME_CTRL_RESETTING) || - !nvme_change_ctrl_state(&ctrl->ctrl, NVME_CTRL_CONNECTING)) { + if (!nvme_change_ctrl_state(&ctrl->ctrl, NVME_CTRL_CONNECTING)) { dev_err(ctrl->ctrl.device, "NVME-FC{%d}: failed to init ctrl state\n", ctrl->cnum); goto fail_ctrl; |