Commit 170b761c authored by Sagi Grimberg's avatar Sagi Grimberg Committed by Luis Henriques

iser-target: Fix flush + disconnect completion handling

commit 128e9cc8 upstream.

ISER_CONN_UP state is not sufficient to know if
we should wait for completion of flush errors and
disconnected_handler event.

Instead, split it to 2 states:
- ISER_CONN_UP: Got to CM connected phase, This state
indicates that we need to wait for a CM disconnect
event before going to teardown.

- ISER_CONN_FULL_FEATURE: Got to full feature phase
after we posted login response, This state indicates
that we posted recv buffers and we need to wait for
flush completions before going to teardown.

Also avoid deffering disconnected handler to a work,
and handle it within disconnected handler.
More work here is needed to handle DEVICE_REMOVAL event
correctly (cleanup all resources).

Squashed:

iser-target: Don't deffer disconnected handler to a work
Signed-off-by: default avatarSagi Grimberg <sagig@mellanox.com>
Signed-off-by: default avatarNicholas Bellinger <nab@linux-iscsi.org>
Signed-off-by: default avatarLuis Henriques <luis.henriques@canonical.com>
parent 96b50563
...@@ -754,6 +754,9 @@ isert_connected_handler(struct rdma_cm_id *cma_id) ...@@ -754,6 +754,9 @@ isert_connected_handler(struct rdma_cm_id *cma_id)
{ {
struct isert_conn *isert_conn = cma_id->context; struct isert_conn *isert_conn = cma_id->context;
pr_info("conn %p\n", isert_conn);
isert_conn->state = ISER_CONN_UP;
kref_get(&isert_conn->conn_kref); kref_get(&isert_conn->conn_kref);
} }
...@@ -780,8 +783,9 @@ isert_put_conn(struct isert_conn *isert_conn) ...@@ -780,8 +783,9 @@ isert_put_conn(struct isert_conn *isert_conn)
* @isert_conn: isert connection struct * @isert_conn: isert connection struct
* *
* Notes: * Notes:
* In case the connection state is UP, move state * In case the connection state is FULL_FEATURE, move state
* to TEMINATING and start teardown sequence (rdma_disconnect). * to TEMINATING and start teardown sequence (rdma_disconnect).
* In case the connection state is UP, complete flush as well.
* *
* This routine must be called with conn_mutex held. Thus it is * This routine must be called with conn_mutex held. Thus it is
* safe to call multiple times. * safe to call multiple times.
...@@ -791,32 +795,31 @@ isert_conn_terminate(struct isert_conn *isert_conn) ...@@ -791,32 +795,31 @@ isert_conn_terminate(struct isert_conn *isert_conn)
{ {
int err; int err;
if (isert_conn->state == ISER_CONN_UP) { switch (isert_conn->state) {
isert_conn->state = ISER_CONN_TERMINATING; case ISER_CONN_TERMINATING:
break;
case ISER_CONN_UP:
/*
* No flush completions will occur as we didn't
* get to ISER_CONN_FULL_FEATURE yet, complete
* to allow teardown progress.
*/
complete(&isert_conn->conn_wait_comp_err);
case ISER_CONN_FULL_FEATURE: /* FALLTHRU */
pr_info("Terminating conn %p state %d\n", pr_info("Terminating conn %p state %d\n",
isert_conn, isert_conn->state); isert_conn, isert_conn->state);
isert_conn->state = ISER_CONN_TERMINATING;
err = rdma_disconnect(isert_conn->conn_cm_id); err = rdma_disconnect(isert_conn->conn_cm_id);
if (err) if (err)
pr_warn("Failed rdma_disconnect isert_conn %p\n", pr_warn("Failed rdma_disconnect isert_conn %p\n",
isert_conn); isert_conn);
break;
default:
pr_warn("conn %p teminating in state %d\n",
isert_conn, isert_conn->state);
} }
} }
static void
isert_disconnect_work(struct work_struct *work)
{
struct isert_conn *isert_conn = container_of(work,
struct isert_conn, conn_logout_work);
pr_debug("isert_disconnect_work(): >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>\n");
mutex_lock(&isert_conn->conn_mutex);
isert_conn_terminate(isert_conn);
mutex_unlock(&isert_conn->conn_mutex);
pr_info("conn %p completing conn_wait\n", isert_conn);
complete(&isert_conn->conn_wait);
}
static int static int
isert_disconnected_handler(struct rdma_cm_id *cma_id) isert_disconnected_handler(struct rdma_cm_id *cma_id)
{ {
...@@ -831,8 +834,12 @@ isert_disconnected_handler(struct rdma_cm_id *cma_id) ...@@ -831,8 +834,12 @@ isert_disconnected_handler(struct rdma_cm_id *cma_id)
isert_conn = (struct isert_conn *)cma_id->context; isert_conn = (struct isert_conn *)cma_id->context;
INIT_WORK(&isert_conn->conn_logout_work, isert_disconnect_work); mutex_lock(&isert_conn->conn_mutex);
schedule_work(&isert_conn->conn_logout_work); isert_conn_terminate(isert_conn);
mutex_unlock(&isert_conn->conn_mutex);
pr_info("conn %p completing conn_wait\n", isert_conn);
complete(&isert_conn->conn_wait);
return 0; return 0;
} }
...@@ -1007,7 +1014,7 @@ isert_init_send_wr(struct isert_conn *isert_conn, struct isert_cmd *isert_cmd, ...@@ -1007,7 +1014,7 @@ isert_init_send_wr(struct isert_conn *isert_conn, struct isert_cmd *isert_cmd,
* bit for every ISERT_COMP_BATCH_COUNT number of ib_post_send() calls. * bit for every ISERT_COMP_BATCH_COUNT number of ib_post_send() calls.
*/ */
mutex_lock(&isert_conn->conn_mutex); mutex_lock(&isert_conn->conn_mutex);
if (coalesce && isert_conn->state == ISER_CONN_UP && if (coalesce && isert_conn->state == ISER_CONN_FULL_FEATURE &&
++isert_conn->conn_comp_batch < ISERT_COMP_BATCH_COUNT) { ++isert_conn->conn_comp_batch < ISERT_COMP_BATCH_COUNT) {
tx_desc->llnode_active = true; tx_desc->llnode_active = true;
llist_add(&tx_desc->comp_llnode, &isert_conn->conn_comp_llist); llist_add(&tx_desc->comp_llnode, &isert_conn->conn_comp_llist);
...@@ -1108,7 +1115,8 @@ isert_put_login_tx(struct iscsi_conn *conn, struct iscsi_login *login, ...@@ -1108,7 +1115,8 @@ isert_put_login_tx(struct iscsi_conn *conn, struct iscsi_login *login,
if (ret) if (ret)
return ret; return ret;
isert_conn->state = ISER_CONN_UP; /* Now we are in FULL_FEATURE phase */
isert_conn->state = ISER_CONN_FULL_FEATURE;
goto post_send; goto post_send;
} }
......
...@@ -23,6 +23,7 @@ enum iser_ib_op_code { ...@@ -23,6 +23,7 @@ enum iser_ib_op_code {
enum iser_conn_state { enum iser_conn_state {
ISER_CONN_INIT, ISER_CONN_INIT,
ISER_CONN_UP, ISER_CONN_UP,
ISER_CONN_FULL_FEATURE,
ISER_CONN_TERMINATING, ISER_CONN_TERMINATING,
ISER_CONN_DOWN, ISER_CONN_DOWN,
}; };
...@@ -138,7 +139,6 @@ struct isert_conn { ...@@ -138,7 +139,6 @@ struct isert_conn {
struct ib_mr *conn_mr; struct ib_mr *conn_mr;
struct ib_qp *conn_qp; struct ib_qp *conn_qp;
struct isert_device *conn_device; struct isert_device *conn_device;
struct work_struct conn_logout_work;
struct mutex conn_mutex; struct mutex conn_mutex;
struct completion conn_wait; struct completion conn_wait;
struct completion conn_wait_comp_err; struct completion conn_wait_comp_err;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment