diff options
| author | Eli Cohen <eli@dev.mellanox.co.il> | 2012-01-03 20:36:48 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2012-01-12 11:29:24 -0800 |
| commit | 585ea9bc6752c41d669ea0f4cee8f3954df326e0 (patch) | |
| tree | 9089e0928ac101795d5d3dab93fc9dbac32d89ce /drivers/atm | |
| parent | 0d878668d0a4663a4807d413532f3a99496eebf4 (diff) | |
IB/uverbs: Protect QP multicast list
commit e214a0fe2b382fa302c036ecd6e6ffe99e3b9875 upstream.
Userspace verbs multicast attach/detach operations on a QP are done
while holding the rwsem of the QP for reading. That's not sufficient
since a reader lock allows more than one reader to acquire the
lock. However, multicast attach/detach does list manipulation that
can corrupt the list if multiple threads run in parallel.
Fix this by acquiring the rwsem as a writer to serialize attach/detach
operations. Add idr_write_qp() and put_qp_write() to encapsulate
this.
This fixes oops seen when running applications that perform multicast
joins/leaves.
Reported by: Mike Dubman <miked@mellanox.com>
Signed-off-by: Eli Cohen <eli@mellanox.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/atm')
0 files changed, 0 insertions, 0 deletions