diff options
author | Arnd Bergmann <arnd@arndb.de> | 2014-04-07 11:33:06 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-04-07 14:24:06 -0400 |
commit | b8780c363d808a726a34793caa900923d32b6b80 (patch) | |
tree | 06e8fe284ccbad76fb2fca0d9e982e5de49bf245 /Documentation/DocBook | |
parent | 240cd6a817bd855e3f1e615ed9ae16407f8cfce6 (diff) |
sched: remove sleep_on() and friends
This is the final piece in the puzzle, as all patches to remove the
last users of \(interruptible_\|\)sleep_on\(_timeout\|\) have made it
into the 3.15 merge window. The work was long overdue, and this
interface in particular should not have survived the BKL removal
that was done a couple of years ago.
Citing Jon Corbet from http://lwn.net/2001/0201/kernel.php3":
"[...] it was suggested that the janitors look for and fix all code
that calls sleep_on() [...] since (1) almost all such code is
incorrect, and (2) Linus has agreed that those functions should
be removed in the 2.5 development series".
We haven't quite made it for 2.5, but maybe we can merge this for 3.15.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'Documentation/DocBook')
-rw-r--r-- | Documentation/DocBook/kernel-hacking.tmpl | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/Documentation/DocBook/kernel-hacking.tmpl b/Documentation/DocBook/kernel-hacking.tmpl index d0758b241b23..bd9015d10cff 100644 --- a/Documentation/DocBook/kernel-hacking.tmpl +++ b/Documentation/DocBook/kernel-hacking.tmpl | |||
@@ -850,16 +850,6 @@ printk(KERN_INFO "my ip: %pI4\n", &ipaddress); | |||
850 | <returnvalue>-ERESTARTSYS</returnvalue> if a signal is received. | 850 | <returnvalue>-ERESTARTSYS</returnvalue> if a signal is received. |
851 | The <function>wait_event()</function> version ignores signals. | 851 | The <function>wait_event()</function> version ignores signals. |
852 | </para> | 852 | </para> |
853 | <para> | ||
854 | Do not use the <function>sleep_on()</function> function family - | ||
855 | it is very easy to accidentally introduce races; almost certainly | ||
856 | one of the <function>wait_event()</function> family will do, or a | ||
857 | loop around <function>schedule_timeout()</function>. If you choose | ||
858 | to loop around <function>schedule_timeout()</function> remember | ||
859 | you must set the task state (with | ||
860 | <function>set_current_state()</function>) on each iteration to avoid | ||
861 | busy-looping. | ||
862 | </para> | ||
863 | 853 | ||
864 | </sect1> | 854 | </sect1> |
865 | 855 | ||