Re: [PATCH v2 01/31] coccinelle: Improve setup_timer.cocci matching

From: Masahiro Yamada
Date: Mon Nov 06 2017 - 19:19:25 EST


2017-09-24 5:43 GMT+09:00 Julia Lawall <julia.lawall@xxxxxxx>:
>
>
> On Wed, 20 Sep 2017, Kees Cook wrote:
>
>> This improves the patch mode of setup_timer.cocci. Several patterns
>> were missing:
>> - assignments-before-init_timer() cases
>> - limit the .data case removal to the specific struct timer_list instance
>> - handling calls by dereference (timer->field vs timer.field)
>>
>> Cc: Julia Lawall <Julia.Lawall@xxxxxxx>
>> Cc: Gilles Muller <Gilles.Muller@xxxxxxx>
>> Cc: Nicolas Palix <nicolas.palix@xxxxxxx>
>> Cc: Michal Marek <mmarek@xxxxxxxx>
>> Cc: cocci@xxxxxxxxxxxxxxx
>> Signed-off-by: Kees Cook <keescook@xxxxxxxxxxxx>
>
> Acked-by: Julia Lawall <julia.lawall@xxxxxxx>
>
> Note that I proposed some changes on this rule as well, on August 23
> (https://systeme.lip6.fr/pipermail/cocci/2017-August/004386.html). My
> changes are still orthogonal to the ones proposed here.
>
> Actually, my changes are in the part about matching, and this patch on
> covers the -D patch case (transformation). The matching rules should be
> extended in the same way that the patch rules are extended below, but it
> would be better to apply my patch first.
>
> julia
>

Applied to linux-kbuild/misc


--
Best Regards
Masahiro Yamada