diff options
| author | Michael S. Tsirkin <mst@mellanox.co.il> | 2006-03-02 22:07:47 +0300 |
|---|---|---|
| committer | Roland Dreier <rolandd@cisco.com> | 2006-03-20 21:08:20 +0300 |
| commit | 9acf6a8570dcfc9f55724b8b71099fc8768e8c26 (patch) | |
| tree | 5b265bf0474f321722d2690b5b5dc57cde80690e /scripts/basic/split-include.c | |
| parent | 54d07e2a1ead2f093ce054cda2e0f5ec163c650c (diff) | |
| download | linux-9acf6a8570dcfc9f55724b8b71099fc8768e8c26.tar.xz | |
IPoIB: Fix multicast race between canceling and completing
ipoib_mcast_stop_thread currently tests mcast->query and if it is
NULL, does not perform wait_for_completion on the mcast and frees the
mcast object directly.
However, since both operations are done without locking, it is
possible that ipoib_mcast_join_complete is in progress on this mcast
object and has set mcast->query to NULL already.
Solve this by:
- taking priv->lock before we change mcast->query in ipoib_mcast_join_complete,
and keeping it until we no longer need the mcast object
- taking priv->lock around mcast->query test in ipoib_mcast_stop_thread
Signed-off-by: Michael S. Tsirkin <mst@mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'scripts/basic/split-include.c')
0 files changed, 0 insertions, 0 deletions