RDMA/cma: Add locking around QP accesses
authorSean Hefty <sean.hefty@intel.com>
Mon, 24 Sep 2007 20:19:09 +0000 (13:19 -0700)
committerRoland Dreier <rolandd@cisco.com>
Tue, 16 Oct 2007 19:25:00 +0000 (12:25 -0700)
commitc5483388bb4d771007ef36478db038e07922a020
treec3e7304a88c0e856d28abaf3219a82e518ea6e9c
parentab8403c424a35364a3a2c753f7c5917fcbb4d809
RDMA/cma: Add locking around QP accesses

If a user allocates a QP on an rdma_cm_id, the rdma_cm will automatically
transition the QP through its states (RTR, RTS, error, etc.)  While the
QP state transitions are occurring, the QP itself must remain valid.
Provide locking around the QP pointer to prevent its destruction while
accessing the pointer.

This fixes an issue reported by Olaf Kirch from Oracle that resulted in
a system crash:

"An incoming connection arrives and we decide to tear down the nascent
 connection.  The remote ends decides to do the same.  We start to shut
 down the connection, and call rdma_destroy_qp on our cm_id. ... Now
 apparently a 'connect reject' message comes in from the other host,
 and cma_ib_handler() is called with an event of IB_CM_REJ_RECEIVED.
 It calls cma_modify_qp_err, which for some odd reason tries to modify
 the exact same QP we just destroyed."

Signed-off-by: Sean Hefty <sean.hefty@intel.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
drivers/infiniband/core/cma.c