summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJoe Perches <joe@perches.com>2012-03-23 23:02:20 +0100
committerLinus Torvalds <torvalds@linux-foundation.org>2012-03-24 00:58:37 +0100
commit2c92488ab29886d08766c054afc8dd0f19c724f1 (patch)
treec1a42d6c825de9b0f3bf239d9a1a086a4b4422ce
parentcheckpatch: add --strict tests for braces, comments and casts (diff)
downloadlinux-2c92488ab29886d08766c054afc8dd0f19c724f1.tar.xz
linux-2c92488ab29886d08766c054afc8dd0f19c724f1.zip
checkpatch: warn on use of yield()
Using yield() is generally wrong. Warn on its use. Signed-off-by: Joe Perches <joe@perches.com> Cc: Andy Whitcroft <apw@canonical.com> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rwxr-xr-xscripts/checkpatch.pl6
1 files changed, 6 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 21486c04b11c..d1fd466043ef 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -3338,6 +3338,12 @@ sub process {
"__func__ should be used instead of gcc specific __FUNCTION__\n" . $herecurr);
}
+# check for use of yield()
+ if ($line =~ /\byield\s*\(\s*\)/) {
+ WARN("YIELD",
+ "Using yield() is generally wrong. See yield() kernel-doc (sched/core.c)\n" . $herecurr);
+ }
+
# check for semaphores initialized locked
if ($line =~ /^.\s*sema_init.+,\W?0\W?\)/) {
WARN("CONSIDER_COMPLETION",