Re: [PATCH v3 12/16] treewide: Use array_size() for kmalloc()-family
From: Julia Lawall
Date: Sun Jul 01 2018 - 05:03:46 EST
> > // 2-factor product with sizeof(variable)
> > @@
> > identifier alloc =~ "kmalloc|kzalloc|kvmalloc|kvzalloc";
>
> * This regular expression could be optimised to the specification âkv?[mz]allocâ.
> Extensions will be useful for further function names.
>
> * The repetition of such a constraint in subsequent SmPL rules could be avoided
> if inheritance will be used for this metavariable.
This is quite incorrect. Inheritance is only possible when a match of the
previous rule has succeeded. If a rule never applies in a given file, the
rules that inherit from it won't apply either. Furthermore, what is
inherited is the value, not the constraint. If the original binding of
alloc only ever matches kmalloc, then the inherited references will only
match kmalloc too.
julia