Skip to content

Commit

Permalink
RDS: Fix send locking issue
Browse files Browse the repository at this point in the history
Fix a deadlock between rds_rdma_send_complete() and
rds_send_remove_from_sock() when rds socket lock and
rds message lock are acquired out-of-order.

Signed-off-by: Tina Yang <Tina.Yang@oracle.com>
Signed-off-by: Andy Grover <andy.grover@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Tina Yang authored and David S. Miller committed Mar 17, 2010
1 parent 2e7b3b9 commit 048c15e
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions net/rds/send.c
Original file line number Diff line number Diff line change
Expand Up @@ -533,14 +533,13 @@ void rds_send_remove_from_sock(struct list_head *messages, int status)

if (rs != rm->m_rs) {
if (rs) {
spin_unlock(&rs->rs_lock);
rds_wake_sk_sleep(rs);
sock_put(rds_rs_to_sk(rs));
}
rs = rm->m_rs;
spin_lock(&rs->rs_lock);
sock_hold(rds_rs_to_sk(rs));
}
spin_lock(&rs->rs_lock);

if (test_and_clear_bit(RDS_MSG_ON_SOCK, &rm->m_flags)) {
struct rds_rdma_op *ro = rm->m_rdma_op;
Expand All @@ -560,14 +559,14 @@ void rds_send_remove_from_sock(struct list_head *messages, int status)
rds_message_put(rm);
rm->m_rs = NULL;
}
spin_unlock(&rs->rs_lock);

unlock_and_drop:
spin_unlock(&rm->m_rs_lock);
rds_message_put(rm);
}

if (rs) {
spin_unlock(&rs->rs_lock);
rds_wake_sk_sleep(rs);
sock_put(rds_rs_to_sk(rs));
}
Expand Down

0 comments on commit 048c15e

Please sign in to comment.