path: root/scripts
diff options
authorJoe Perches <>2011-06-15 15:08:17 -0700
committerLinus Torvalds <>2011-06-15 20:03:59 -0700
commit17441227f6258fc379c6ebfe21c3eec43b6f0de3 (patch)
treed38f88d97472ae33827fd8cd247e0cd5161c61ba /scripts
parentac5622418bbff9cd3dc607aa57dfb4f62a7f2043 (diff)
checkpatch: add warning for uses of printk_ratelimit
Warn about uses of printk_ratelimit() because it uses a global state and can hide subsequent useful messages. Signed-off-by: Joe Perches <> Cc: Andy Whitcroft <> Cc: Richard Weinberger <> Signed-off-by: Andrew Morton <> Signed-off-by: Linus Torvalds <>
Diffstat (limited to 'scripts')
1 files changed, 5 insertions, 0 deletions
diff --git a/scripts/ b/scripts/
index 8657f99bfb2b..b0aa2c680593 100755
--- a/scripts/
+++ b/scripts/
@@ -1943,6 +1943,11 @@ sub process {
WARN("LINUX_VERSION_CODE should be avoided, code should be for the version to which it is merged\n" . $herecurr);
+# check for uses of printk_ratelimit
+ if ($line =~ /\bprintk_ratelimit\s*\(/) {
+ WARN("Prefer printk_ratelimited or pr_<level>_ratelimited to printk_ratelimit\n" . $herecurr);
+ }
# printk should use KERN_* levels. Note that follow on printk's on the
# same line do not need a level, so we use the current block context
# to try and find and validate the current printk. In summary the current