diff options
author | Alex Elder <elder@dreamhost.com> | 2012-04-04 22:35:44 +0400 |
---|---|---|
committer | Alex Elder <elder@dreamhost.com> | 2012-04-06 00:43:58 +0400 |
commit | cd9d9f5df6098c50726200d4185e9e8da32785b3 (patch) | |
tree | 94e4c7035fc7c7f6596a8f3b59fa27e19a45d4f0 /fs/ceph/ioctl.c | |
parent | c666601a935b94cc0f3310339411b6940de751ba (diff) | |
download | linux-cd9d9f5df6098c50726200d4185e9e8da32785b3.tar.xz |
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 'fs/ceph/ioctl.c')
0 files changed, 0 insertions, 0 deletions