diff options
author | Steve Wise <swise@opengridcomputing.com> | 2013-04-23 16:17:14 (GMT) |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2013-04-25 00:47:33 (GMT) |
commit | e413a823f60b582af471f0079eb99f50d34b0da7 (patch) | |
tree | 945c3a57df72e353f03a5ad3b7623df219dcab56 | |
parent | cc529c0d72bd2490801ce324caf2ea0a0c1d7b1e (diff) | |
download | linux-e413a823f60b582af471f0079eb99f50d34b0da7.tar.xz |
RDMA/iwcm: Don't touch cmid after dropping reference
The function cm_work_handler() cannot touch the cm_id after it derefs
it, because it might be freed on another concurrent thread. If there
are more work items queued for this cm_id, then we know there must be
more references because they are added when the work items are queued.
So in the while loop inside cm_work_handler(), after derefing, if the
queue is empty, then exit the function. Otherwise we know it's safe
to re-acquire the lock.
Signed-off-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
-rw-r--r-- | drivers/infiniband/core/iwcm.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/infiniband/core/iwcm.c b/drivers/infiniband/core/iwcm.c index 0bb99bb..c47c203 100644 --- a/drivers/infiniband/core/iwcm.c +++ b/drivers/infiniband/core/iwcm.c @@ -878,6 +878,8 @@ static void cm_work_handler(struct work_struct *_work) } return; } + if (empty) + return; spin_lock_irqsave(&cm_id_priv->lock, flags); } spin_unlock_irqrestore(&cm_id_priv->lock, flags); |