diff options
author | Or Gerlitz <ogerlitz@voltaire.com> | 2010-05-05 10:30:34 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2010-05-12 12:30:44 -0400 |
commit | d265b9808272c9f25e1c36d3fb5ddb466efd90e9 (patch) | |
tree | 65791c6f7b3f25e275a4b145795b3c5041b764f8 /drivers | |
parent | 2110f9bf37511df06220bb7e977f417baecf2950 (diff) |
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>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/infiniband/ulp/iser/iser_verbs.c | 2 |
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 b9d6aa102aa5..ed7c90135412 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) | |||
346 | /* on EVENT_ADDR_ERROR there's no device yet for this conn */ | 346 | /* on EVENT_ADDR_ERROR there's no device yet for this conn */ |
347 | if (device != NULL) | 347 | if (device != NULL) |
348 | iser_device_try_release(device); | 348 | iser_device_try_release(device); |
349 | if (ib_conn->iser_conn) | ||
350 | ib_conn->iser_conn->ib_conn = NULL; | ||
351 | iscsi_destroy_endpoint(ib_conn->ep); | 349 | iscsi_destroy_endpoint(ib_conn->ep); |
352 | } | 350 | } |
353 | 351 | ||