Commit 4a579da2 authored by Sagi Grimberg's avatar Sagi Grimberg Committed by Nicholas Bellinger

iser-target: Fix possible deadlock in RDMA_CM connection error

Before we reach to connection established we may get an
error event. In this case the core won't teardown this
connection (never established it), so we take care of freeing
it ourselves.
Signed-off-by: default avatarSagi Grimberg <sagig@mellanox.com>
Cc: <stable@vger.kernel.org> # v3.10+
Signed-off-by: default avatarNicholas Bellinger <nab@linux-iscsi.org>
parent 364189f0
...@@ -222,7 +222,7 @@ isert_alloc_rx_descriptors(struct isert_conn *isert_conn) ...@@ -222,7 +222,7 @@ isert_alloc_rx_descriptors(struct isert_conn *isert_conn)
static void static void
isert_free_rx_descriptors(struct isert_conn *isert_conn) isert_free_rx_descriptors(struct isert_conn *isert_conn)
{ {
struct ib_device *ib_dev = isert_conn->conn_cm_id->device; struct ib_device *ib_dev = isert_conn->conn_device->ib_device;
struct iser_rx_desc *rx_desc; struct iser_rx_desc *rx_desc;
int i; int i;
...@@ -719,8 +719,8 @@ isert_connect_request(struct rdma_cm_id *cma_id, struct rdma_cm_event *event) ...@@ -719,8 +719,8 @@ isert_connect_request(struct rdma_cm_id *cma_id, struct rdma_cm_event *event)
static void static void
isert_connect_release(struct isert_conn *isert_conn) isert_connect_release(struct isert_conn *isert_conn)
{ {
struct ib_device *ib_dev = isert_conn->conn_cm_id->device;
struct isert_device *device = isert_conn->conn_device; struct isert_device *device = isert_conn->conn_device;
struct ib_device *ib_dev = device->ib_device;
isert_dbg("conn %p\n", isert_conn); isert_dbg("conn %p\n", isert_conn);
...@@ -728,6 +728,7 @@ isert_connect_release(struct isert_conn *isert_conn) ...@@ -728,6 +728,7 @@ isert_connect_release(struct isert_conn *isert_conn)
isert_conn_free_fastreg_pool(isert_conn); isert_conn_free_fastreg_pool(isert_conn);
isert_free_rx_descriptors(isert_conn); isert_free_rx_descriptors(isert_conn);
if (isert_conn->conn_cm_id)
rdma_destroy_id(isert_conn->conn_cm_id); rdma_destroy_id(isert_conn->conn_cm_id);
if (isert_conn->conn_qp) { if (isert_conn->conn_qp) {
...@@ -878,12 +879,15 @@ isert_disconnected_handler(struct rdma_cm_id *cma_id, ...@@ -878,12 +879,15 @@ isert_disconnected_handler(struct rdma_cm_id *cma_id,
return 0; return 0;
} }
static void static int
isert_connect_error(struct rdma_cm_id *cma_id) isert_connect_error(struct rdma_cm_id *cma_id)
{ {
struct isert_conn *isert_conn = cma_id->qp->qp_context; struct isert_conn *isert_conn = cma_id->qp->qp_context;
isert_conn->conn_cm_id = NULL;
isert_put_conn(isert_conn); isert_put_conn(isert_conn);
return -1;
} }
static int static int
...@@ -912,7 +916,7 @@ isert_cma_handler(struct rdma_cm_id *cma_id, struct rdma_cm_event *event) ...@@ -912,7 +916,7 @@ isert_cma_handler(struct rdma_cm_id *cma_id, struct rdma_cm_event *event)
case RDMA_CM_EVENT_REJECTED: /* FALLTHRU */ case RDMA_CM_EVENT_REJECTED: /* FALLTHRU */
case RDMA_CM_EVENT_UNREACHABLE: /* FALLTHRU */ case RDMA_CM_EVENT_UNREACHABLE: /* FALLTHRU */
case RDMA_CM_EVENT_CONNECT_ERROR: case RDMA_CM_EVENT_CONNECT_ERROR:
isert_connect_error(cma_id); ret = isert_connect_error(cma_id);
break; break;
default: default:
isert_err("Unhandled RDMA CMA event: %d\n", event->event); isert_err("Unhandled RDMA CMA event: %d\n", event->event);
......
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