summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOr Gerlitz <ogerlitz@voltaire.com>2010-05-05 17:30:34 +0300
committerRoland Dreier <rolandd@cisco.com>2010-05-12 09:30:44 -0700
commitd265b9808272c9f25e1c36d3fb5ddb466efd90e9 (patch)
tree65791c6f7b3f25e275a4b145795b3c5041b764f8
parent2110f9bf37511df06220bb7e977f417baecf2950 (diff)
downloadop-kernel-dev-d265b9808272c9f25e1c36d3fb5ddb466efd90e9.zip
op-kernel-dev-d265b9808272c9f25e1c36d3fb5ddb466efd90e9.tar.gz
IB/iser: Remove buggy back-pointer setting
The iscsi connection object life cycle includes binding and unbinding (conn_stop) to/from the iscsi transport connection object. Since iscsi connection objects are recycled, at the time the transport connection (e.g iser's IB connection) is released, it is not valid to touch the iscsi connection tied to the transport back-pointer since it may already point to a different transport connection. Signed-off-by: Or Gerlitz <ogerlitz@voltaire.com> Signed-off-by: Roland Dreier <rolandd@cisco.com>
-rw-r--r--drivers/infiniband/ulp/iser/iser_verbs.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/infiniband/ulp/iser/iser_verbs.c b/drivers/infiniband/ulp/iser/iser_verbs.c
index b9d6aa1..ed7c901 100644
--- a/drivers/infiniband/ulp/iser/iser_verbs.c
+++ b/drivers/infiniband/ulp/iser/iser_verbs.c
@@ -346,8 +346,6 @@ static void iser_conn_release(struct iser_conn *ib_conn)
/* on EVENT_ADDR_ERROR there's no device yet for this conn */
if (device != NULL)
iser_device_try_release(device);
- if (ib_conn->iser_conn)
- ib_conn->iser_conn->ib_conn = NULL;
iscsi_destroy_endpoint(ib_conn->ep);
}
OpenPOWER on IntegriCloud