lkml.org 
[lkml]   [2017]   [Aug]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Coccinelle: add atomic_as_refcounter script
> +identifier fname =~ ".*free.*";
> +identifier fname2 =~ ".*destroy.*";
> +identifier fname3 =~ ".*del.*";
> +identifier fname4 =~ ".*queue_work.*";
> +identifier fname5 =~ ".*schedule_work.*";
> +identifier fname6 =~ ".*call_rcu.*";

Personally, I find the above regular expressions much easier to understand
than the merged version that Markus proposed. But the performance issue is
only on whether to use regular expressions or not. If you use regular
expressions, Coccinelle will not do some optimizations. But once you
decide to use regular expressions, the performance hit is already taken -
for a good cause here, to my understanding. So just put whatever you find
convenient, in terms of readability, precision, etc. It seems that del is
not very precise, because it is a substring of multiple words with
different meanings. Maybe it should be improved, or maybe one can just
live with the false positives (eg delay), if they actually are false
positives.

julia

\
 
 \ /
  Last update: 2017-08-17 13:51    [W:0.055 / U:0.360 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site