diff options
| author | Paul Walmsley <pjw@kernel.org> | 2026-08-06 19:00:47 -0600 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2026-08-13 15:42:09 -0700 |
| commit | 5779e0f30fa4246748d4168a0f53298443f3744e (patch) | |
| tree | 65bb9ed7e0ddae24d631645f5a6e822f8bbe1ea5 /scripts/checkpatch.pl | |
| parent | e300eb5002925b29be803d2661af07266cfa267e (diff) | |
| download | linux-5779e0f30fa4246748d4168a0f53298443f3744e.tar.gz linux-5779e0f30fa4246748d4168a0f53298443f3744e.zip | |
checkpatch: add NOKPROBE_SYMBOL to the whitelist of lines that can occur immediately after functions
It's customary for NOKPROBE_SYMBOL() macro usage to appear immediately
after a function's final closing brace, but checkpatch doesn't know that
yet. As a result, checkpatch --strict incorrectly flags this common
kernel pattern, e.g.,
CHECK: Please use a blank line after function/struct/union/enum declarations
33: FILE: arch/riscv/kernel/traps.c:273:
}
+NOKPROBE_SYMBOL(probe_single_step_handler);
Fix by adding NOKPROBE_SYMBOL to the whitelist of patterns that are
cleared to appear immediately after functions.
Link: https://lore.kernel.org/130be7db-6098-86a4-60fe-0c1a5d9e30ba@kernel.org
Signed-off-by: Paul Walmsley <pjw@kernel.org>
Acked-by: Joe Perches <joe@perches.com>
Cc: Nam Cao <namcao@linutronix.de>
Cc: Jisheng Zhang <jszhang@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'scripts/checkpatch.pl')
| -rwxr-xr-x | scripts/checkpatch.pl | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 7a846a3ea127..73b5e788e84e 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -4154,6 +4154,7 @@ sub process { $line =~ /^\+[a-z_]*init/ || $line =~ /^\+\s*(?:static\s+)?[A-Z_]*ATTR/ || $line =~ /^\+\s*DECLARE/ || + $line =~ /^\+\s*NOKPROBE_SYMBOL/ || $line =~ /^\+\s*builtin_[\w_]*driver/ || $line =~ /^\+\s*__setup/)) { if (CHK("LINE_SPACING", |
