diff options
author | Andrew Morton <akpm@linux-foundation.org> | 2010-10-26 17:22:49 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-10-26 19:52:16 -0400 |
commit | 77006a0a828249dd69341f960043ee41e7487aa0 (patch) | |
tree | e3524692543f73fe5c75f5aefad48fecf4cdb4bb /include/linux/kernel.h | |
parent | f5d87d851d76a390d0fab2f77bd1d563d69ee586 (diff) |
ratelimit: add comment warning people off printk_ratelimit()
printk_ratelimit() was a bad idea - we don't want subsytem A causing
ratelimiting of subsystem B's messages.
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include/linux/kernel.h')
-rw-r--r-- | include/linux/kernel.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/include/linux/kernel.h b/include/linux/kernel.h index e9b492b33032..77b04ed037df 100644 --- a/include/linux/kernel.h +++ b/include/linux/kernel.h | |||
@@ -277,6 +277,11 @@ asmlinkage int vprintk(const char *fmt, va_list args) | |||
277 | asmlinkage int printk(const char * fmt, ...) | 277 | asmlinkage int printk(const char * fmt, ...) |
278 | __attribute__ ((format (printf, 1, 2))) __cold; | 278 | __attribute__ ((format (printf, 1, 2))) __cold; |
279 | 279 | ||
280 | /* | ||
281 | * Please don't use printk_ratelimit(), because it shares ratelimiting state | ||
282 | * with all other unrelated printk_ratelimit() callsites. Instead use | ||
283 | * printk_ratelimited() or plain old __ratelimit(). | ||
284 | */ | ||
280 | extern int __printk_ratelimit(const char *func); | 285 | extern int __printk_ratelimit(const char *func); |
281 | #define printk_ratelimit() __printk_ratelimit(__func__) | 286 | #define printk_ratelimit() __printk_ratelimit(__func__) |
282 | extern bool printk_timed_ratelimit(unsigned long *caller_jiffies, | 287 | extern bool printk_timed_ratelimit(unsigned long *caller_jiffies, |