Commit 715a588f authored by Krishna Kumar's avatar Krishna Kumar Committed by Roland Dreier

RDMA/iwcm: Remove unnecessary function argument

Remove unnecessary cm_id_priv argument to copy_private_data(), and
change text to reflect the code.  Fix couple of typos in comments.
Signed-off-by: default avatarKrishna Kumar <krkumar2@in.ibm.com>
Acked-by: default avatarSteve Wise <swise@opengridcomputing.com>
Signed-off-by: default avatarRoland Dreier <rolandd@cisco.com>
parent 13fccdb3
...@@ -80,7 +80,7 @@ struct iwcm_work { ...@@ -80,7 +80,7 @@ struct iwcm_work {
* 1) in the event upcall, cm_event_handler(), for a listening cm_id. If * 1) in the event upcall, cm_event_handler(), for a listening cm_id. If
* the backlog is exceeded, then no more connection request events will * the backlog is exceeded, then no more connection request events will
* be processed. cm_event_handler() returns -ENOMEM in this case. Its up * be processed. cm_event_handler() returns -ENOMEM in this case. Its up
* to the provider to reject the connectino request. * to the provider to reject the connection request.
* 2) in the connection request workqueue handler, cm_conn_req_handler(). * 2) in the connection request workqueue handler, cm_conn_req_handler().
* If work elements cannot be allocated for the new connect request cm_id, * If work elements cannot be allocated for the new connect request cm_id,
* then IWCM will call the provider reject method. This is ok since * then IWCM will call the provider reject method. This is ok since
...@@ -131,12 +131,11 @@ static int alloc_work_entries(struct iwcm_id_private *cm_id_priv, int count) ...@@ -131,12 +131,11 @@ static int alloc_work_entries(struct iwcm_id_private *cm_id_priv, int count)
} }
/* /*
* Save private data from incoming connection requests in the * Save private data from incoming connection requests to
* cm_id_priv so the low level driver doesn't have to. Adjust * iw_cm_event, so the low level driver doesn't have to. Adjust
* the event ptr to point to the local copy. * the event ptr to point to the local copy.
*/ */
static int copy_private_data(struct iwcm_id_private *cm_id_priv, static int copy_private_data(struct iw_cm_event *event)
struct iw_cm_event *event)
{ {
void *p; void *p;
...@@ -242,7 +241,7 @@ static int iwcm_modify_qp_sqd(struct ib_qp *qp) ...@@ -242,7 +241,7 @@ static int iwcm_modify_qp_sqd(struct ib_qp *qp)
/* /*
* CM_ID <-- CLOSING * CM_ID <-- CLOSING
* *
* Block if a passive or active connection is currenlty being processed. Then * Block if a passive or active connection is currently being processed. Then
* process the event as follows: * process the event as follows:
* - If we are ESTABLISHED, move to CLOSING and modify the QP state * - If we are ESTABLISHED, move to CLOSING and modify the QP state
* based on the abrupt flag * based on the abrupt flag
...@@ -907,7 +906,7 @@ static int cm_event_handler(struct iw_cm_id *cm_id, ...@@ -907,7 +906,7 @@ static int cm_event_handler(struct iw_cm_id *cm_id,
if ((work->event.event == IW_CM_EVENT_CONNECT_REQUEST || if ((work->event.event == IW_CM_EVENT_CONNECT_REQUEST ||
work->event.event == IW_CM_EVENT_CONNECT_REPLY) && work->event.event == IW_CM_EVENT_CONNECT_REPLY) &&
work->event.private_data_len) { work->event.private_data_len) {
ret = copy_private_data(cm_id_priv, &work->event); ret = copy_private_data(&work->event);
if (ret) { if (ret) {
put_work(work); put_work(work);
goto out; goto out;
......
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