diff options
| author | Marco Elver <elver@google.com> | 2020-04-01 12:17:14 +0200 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@kernel.org> | 2020-05-06 19:54:58 +0200 |
| commit | 5099a722e9727fe9a93fac51e961735f40e5b6c8 (patch) | |
| tree | 48dd8039040943c781b8c9e984ece042dc1faf5d /scripts | |
| parent | kcsan: Use GFP_ATOMIC under spin lock (diff) | |
| download | linux-5099a722e9727fe9a93fac51e961735f40e5b6c8.tar.xz linux-5099a722e9727fe9a93fac51e961735f40e5b6c8.zip | |
checkpatch: Warn about data_race() without comment
Warn about applications of data_race() without a comment, to encourage
documenting the reasoning behind why it was deemed safe.
Suggested-by: Will Deacon <will@kernel.org>
Signed-off-by: Marco Elver <elver@google.com>
Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/checkpatch.pl | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index a63380c6b0d2..48bb9508e300 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -5833,6 +5833,14 @@ sub process { } } +# check for data_race without a comment. + if ($line =~ /\bdata_race\s*\(/) { + if (!ctx_has_comment($first_line, $linenr)) { + WARN("DATA_RACE", + "data_race without comment\n" . $herecurr); + } + } + # check for smp_read_barrier_depends and read_barrier_depends if (!$file && $line =~ /\b(smp_|)read_barrier_depends\s*\(/) { WARN("READ_BARRIER_DEPENDS", |