diff options
| author |   Alex Elder <elder@dreamhost.com> | 2012-04-04 13:35:44 -0500 |
|---|---|---|
| committer | Alex Elder <elder@dreamhost.com> | 2012-04-05 15:43:58 -0500 |
| commit | cd9d9f5df6098c50726200d4185e9e8da32785b3 (patch) | |
| tree | 94e4c7035fc7c7f6596a8f3b59fa27e19a45d4f0 /include/linux/ceph | |
| parent | rbd: move snap_rwsem to the device, rename to header_rwsem (diff) | |
| download | linux-dev-cd9d9f5df6098c50726200d4185e9e8da32785b3.tar.xz linux-dev-cd9d9f5df6098c50726200d4185e9e8da32785b3.zip | |
rbd: don't hold spinlock during messenger flush
A recent change made changes to the rbd_client_list be protected by
a spinlock. Unfortunately in rbd_put_client(), the lock is taken
before possibly dropping the last reference to an rbd_client, and on
the last reference that eventually calls flush_workqueue() which can
sleep.
The problem was flagged by a debug spinlock warning:
BUG: spinlock wrong CPU on CPU#3, rbd/27814
The solution is to move the spinlock acquisition and release inside
rbd_client_release(), which is the spot where it's really needed for
protecting the removal of the rbd_client from the client list.
Signed-off-by: Alex Elder <elder@dreamhost.com>
Reviewed-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'include/linux/ceph')
0 files changed, 0 insertions, 0 deletions