diff options
author | Steve Wise <swise@opengridcomputing.com> | 2007-03-05 18:32:46 -0500 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2007-03-06 14:47:05 -0500 |
commit | 7d526e6b2c5d6bba70fdc1fc2943bdaf9cc6147d (patch) | |
tree | 8a83a69c8ce7698eae54b40f9371683c676837c7 /drivers/infiniband | |
parent | 88171cfed5810a2354eb1977883589a05ce8d304 (diff) |
RDMA/cxgb3: Start ep timer on a MPA reject
If the consumer rejects the connection we end up under-referencing the
endpoint structure. The fix is to call iwch_ep_disconnect() instead
of the low level disconnect functions so that the endpoint close timer
is started correctly.
Signed-off-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband')
-rw-r--r-- | drivers/infiniband/hw/cxgb3/iwch_cm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/infiniband/hw/cxgb3/iwch_cm.c b/drivers/infiniband/hw/cxgb3/iwch_cm.c index b21fde8b659d..135f80044de5 100644 --- a/drivers/infiniband/hw/cxgb3/iwch_cm.c +++ b/drivers/infiniband/hw/cxgb3/iwch_cm.c | |||
@@ -1687,12 +1687,11 @@ int iwch_reject_cr(struct iw_cm_id *cm_id, const void *pdata, u8 pdata_len) | |||
1687 | return -ECONNRESET; | 1687 | return -ECONNRESET; |
1688 | } | 1688 | } |
1689 | BUG_ON(state_read(&ep->com) != MPA_REQ_RCVD); | 1689 | BUG_ON(state_read(&ep->com) != MPA_REQ_RCVD); |
1690 | state_set(&ep->com, CLOSING); | ||
1691 | if (mpa_rev == 0) | 1690 | if (mpa_rev == 0) |
1692 | abort_connection(ep, NULL, GFP_KERNEL); | 1691 | abort_connection(ep, NULL, GFP_KERNEL); |
1693 | else { | 1692 | else { |
1694 | err = send_mpa_reject(ep, pdata, pdata_len); | 1693 | err = send_mpa_reject(ep, pdata, pdata_len); |
1695 | err = send_halfclose(ep, GFP_KERNEL); | 1694 | err = iwch_ep_disconnect(ep, 0, GFP_KERNEL); |
1696 | } | 1695 | } |
1697 | return 0; | 1696 | return 0; |
1698 | } | 1697 | } |