diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2005-10-30 23:02:47 (GMT) |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-10-31 01:37:23 (GMT) |
commit | 2384f55f8aa520172c995965bd2f8a9740d53095 (patch) | |
tree | 3f2f746529a760824d87f790418bacb94cc5a247 | |
parent | 7407251a0e2ed099e4b12b742b635503e981507c (diff) | |
download | linux-2384f55f8aa520172c995965bd2f8a9740d53095.tar.xz |
[PATCH] coredump_wait() cleanup
This patch deletes pointless code from coredump_wait().
1. It does useless mm->core_waiters inc/dec under mm->mmap_sem,
but any changes to ->core_waiters have no effect until we drop
->mmap_sem.
2. It calls yield() for absolutely unknown reason.
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | fs/exec.c | 13 |
1 files changed, 5 insertions, 8 deletions
@@ -1417,19 +1417,16 @@ static void zap_threads (struct mm_struct *mm) static void coredump_wait(struct mm_struct *mm) { DECLARE_COMPLETION(startup_done); + int core_waiters; - mm->core_waiters++; /* let other threads block */ mm->core_startup_done = &startup_done; - /* give other threads a chance to run: */ - yield(); - zap_threads(mm); - if (--mm->core_waiters) { - up_write(&mm->mmap_sem); + core_waiters = mm->core_waiters; + up_write(&mm->mmap_sem); + + if (core_waiters) wait_for_completion(&startup_done); - } else - up_write(&mm->mmap_sem); BUG_ON(mm->core_waiters); } |