diff options
author | Shaohua Li <shli@fb.com> | 2016-02-25 01:41:01 (GMT) |
---|---|---|
committer | Shaohua Li <shli@fb.com> | 2016-02-26 17:44:57 (GMT) |
commit | 70d9798b95562abac005d4ba71d28820f9a201eb (patch) | |
tree | 4ab92bc060a79faa8363f29462cea00cbbddcf36 /drivers | |
parent | 399146b80ed6fb9f1ebe5a07234f00dff446d2b4 (diff) | |
download | linux-70d9798b95562abac005d4ba71d28820f9a201eb.tar.xz |
MD: warn for potential deadlock
The personality thread shouldn't call mddev_suspend(). Because
mddev_suspend() will for all IO finish, but IO is handled in personality
thread, so this could cause deadlock. To trigger this early, add a
warning if mddev_suspend() is called from personality thread.
Suggested-by: NeilBrown <neilb@suse.com>
Cc: Artur Paszkiewicz <artur.paszkiewicz@intel.com>
Signed-off-by: Shaohua Li <shli@fb.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/md/md.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 464627b..c068f17 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -305,6 +305,7 @@ static blk_qc_t md_make_request(struct request_queue *q, struct bio *bio) */ void mddev_suspend(struct mddev *mddev) { + WARN_ON_ONCE(current == mddev->thread->tsk); if (mddev->suspended++) return; synchronize_rcu(); |