diff options
author | Oleg Nesterov <oleg@redhat.com> | 2009-06-19 02:51:13 +0200 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-07-27 12:15:46 -0700 |
commit | 9ae260270c90643156cda73427aa1f04c923e627 (patch) | |
tree | d69e5b2e0863a34fe9ba0a7a10c24c3d42276d51 /kernel/kthread.c | |
parent | 6560dc160f3a96b8f1f43e2c6b51aa6eb9898b90 (diff) | |
download | linux-stable-9ae260270c90643156cda73427aa1f04c923e627.tar.gz linux-stable-9ae260270c90643156cda73427aa1f04c923e627.tar.bz2 linux-stable-9ae260270c90643156cda73427aa1f04c923e627.zip |
update the comment in kthread_stop()
Commit 63706172f332fd3f6e7458ebfb35fa6de9c21dc5 ("kthreads: rework
kthread_stop()") removed the limitation that the thread function mysr
not call do_exit() itself, but forgot to update the comment.
Since that commit it is OK to use kthread_stop() even if kthread can
exit itself.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel/kthread.c')
-rw-r--r-- | kernel/kthread.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/kernel/kthread.c b/kernel/kthread.c index 9b1a7de26979..eb8751aa0418 100644 --- a/kernel/kthread.c +++ b/kernel/kthread.c @@ -180,10 +180,12 @@ EXPORT_SYMBOL(kthread_bind); * @k: thread created by kthread_create(). * * Sets kthread_should_stop() for @k to return true, wakes it, and - * waits for it to exit. Your threadfn() must not call do_exit() - * itself if you use this function! This can also be called after - * kthread_create() instead of calling wake_up_process(): the thread - * will exit without calling threadfn(). + * waits for it to exit. This can also be called after kthread_create() + * instead of calling wake_up_process(): the thread will exit without + * calling threadfn(). + * + * If threadfn() may call do_exit() itself, the caller must ensure + * task_struct can't go away. * * Returns the result of threadfn(), or %-EINTR if wake_up_process() * was never called. |