Re: [PATCH] checkpatch: Add TP_printk to list of logging functions

From: Song Liu
Date: Wed Oct 18 2017 - 12:31:42 EST



> On Oct 17, 2017, at 4:29 PM, Joe Perches <joe@xxxxxxxxxxx> wrote:
>
> So the line length check can be bypassed by its callers.
>
> Reported-by: Song Liu <songliubraving@xxxxxx>
> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
> ---
> scripts/checkpatch.pl | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 2a8c6c3c1bdb..359c02b0954e 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -454,6 +454,7 @@ our $zero_initializer = qr{(?:(?:0[xX])?0+$Int_type?|NULL|false)\b};
> our $logFunctions = qr{(?x:
> printk(?:_ratelimited|_once|_deferred_once|_deferred|)|
> (?:[a-z0-9]+_){1,2}(?:printk|emerg|alert|crit|err|warning|warn|notice|info|debug|dbg|vdbg|devel|cont|WARN)(?:_ratelimited|_once|)|
> + TP_printk|
> WARN(?:_RATELIMIT|_ONCE|)|
> panic|
> MODULE_[A-Z_]+|
> --
> 2.10.0.rc2.1.g053435c
>

Tested-by: Song Liu <songliubraving@xxxxxx>

Thanks for the fix!