lkml.org 
[lkml]   [2014]   [Sep]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 1/2] Return a value from printk_ratelimited
On Fri, 19 Sep 2014 02:01:29 -0700
Omar Sandoval <osandov@osandov.com> wrote:

> printk returns an integer; there's no reason for printk_ratelimited to swallow
> it.
>
> Signed-off-by: Omar Sandoval <osandov@osandov.com>
> ---
> include/linux/printk.h | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/include/linux/printk.h b/include/linux/printk.h
> index d78125f..67534bc 100644
> --- a/include/linux/printk.h
> +++ b/include/linux/printk.h
> @@ -343,12 +343,14 @@ extern asmlinkage void dump_stack(void) __cold;
> #ifdef CONFIG_PRINTK
> #define printk_ratelimited(fmt, ...) \
> ({ \
> + int __ret = 0; \

My only issues is with the "__ret" name.

It's not really unique enough. If something else uses __ret and does

printk_ratelimit("some fmt string %d\n", __ret);

This will not print the right value.

printk_ratelimit can be used almost anywhere thus using a really unique
value may be worth while here.

What about:

int ______r

?

-- Steve

> static DEFINE_RATELIMIT_STATE(_rs, \
> DEFAULT_RATELIMIT_INTERVAL, \
> DEFAULT_RATELIMIT_BURST); \
> \
> if (__ratelimit(&_rs)) \
> - printk(fmt, ##__VA_ARGS__); \
> + __ret = printk(fmt, ##__VA_ARGS__); \
> + __ret; \
> })
> #else
> #define printk_ratelimited(fmt, ...) \



\
 
 \ /
  Last update: 2014-09-19 20:01    [W:0.601 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site