diff options
author | Andrew Morton <akpm@osdl.org> | 2006-06-23 02:05:15 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-23 07:43:03 -0700 |
commit | 0ae26f1b3159f3acb21ae1e866c3c7e16edd450f (patch) | |
tree | 28a86cc8682ad5b247f597773572e410b9457d72 /kernel/fork.c | |
parent | 32797f976d62af709943aae38a9457b5a6dbcd8a (diff) | |
download | blackbird-op-linux-0ae26f1b3159f3acb21ae1e866c3c7e16edd450f.tar.gz blackbird-op-linux-0ae26f1b3159f3acb21ae1e866c3c7e16edd450f.zip |
[PATCH] mmput() might sleep
exit_aio() and exit_mmap() can sleep. But it's easy to accidentally call
mmput() from inside locks.
Cc: Dave Peterson <dsp@llnl.gov>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'kernel/fork.c')
-rw-r--r-- | kernel/fork.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/fork.c b/kernel/fork.c index ac8100e3088a..195958a3a4d6 100644 --- a/kernel/fork.c +++ b/kernel/fork.c @@ -368,6 +368,8 @@ void fastcall __mmdrop(struct mm_struct *mm) */ void mmput(struct mm_struct *mm) { + might_sleep(); + if (atomic_dec_and_test(&mm->mm_users)) { exit_aio(mm); exit_mmap(mm); |