diff options
| author | Joe Perches <joe@perches.com> | 2018-09-04 18:46:20 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2018-09-04 19:45:02 -0400 |
| commit | c5967e989f1fe702e75d7405b9251ec7e490d847 (patch) | |
| tree | d083551d8f67cdde55f4c31c06b0a59762580f9d /scripts | |
| parent | 62ec0d8c4f332dedf19d6fad15ddea639044d5fe (diff) | |
checkpatch: add __ro_after_init to known $Attribute
__ro_after_init is a specific __attribute__ that checkpatch does currently
not understand.
Add it to the known $Attribute types so that code that uses variables
declared with __ro_after_init are not thought to be a modifier type.
This appears as a defect in checkpatch output of code like:
static bool trust_cpu __ro_after_init = IS_ENABLED(CONFIG_RANDOM_TRUST_CPU);
[...]
if (trust_cpu && arch_init) {
where checkpatch reports:
ERROR: space prohibited after that '&&' (ctx:WxW)
if (trust_cpu && arch_init) {
Link: http://lkml.kernel.org/r/0fa8a2cb83ade4c525e18261ecf6cfede3015983.camel@perches.com
Signed-off-by: Joe Perches <joe@perches.com>
Reported-by: Kees Cook <keescook@chromium.org>
Tested-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts')
| -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 b4caee6e269c..161b0224d6ae 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl | |||
| @@ -380,6 +380,7 @@ our $Attribute = qr{ | |||
| 380 | __noclone| | 380 | __noclone| |
| 381 | __deprecated| | 381 | __deprecated| |
| 382 | __read_mostly| | 382 | __read_mostly| |
| 383 | __ro_after_init| | ||
| 383 | __kprobes| | 384 | __kprobes| |
| 384 | $InitAttribute| | 385 | $InitAttribute| |
| 385 | ____cacheline_aligned| | 386 | ____cacheline_aligned| |
