On Mon, Sep 3, 2018 at 12:10 AM Vlad Buslov <vla...@mellanox.com> wrote:
>
> Recent refactoring of add_metainfo() caused use_all_metadata() to add
> metainfo to ife action metalist without taking reference to module. This
> causes warning in module_put called from ife action cleanup function.
>
> Implement add_metainfo_and_get_ops() function that returns with reference
> to module taken if metainfo was added successfully, and call it from
> use_all_metadata(), instead of calling __add_metainfo() directly.
>
...
>
> Fixes: 5ffe57da29b3 ("act_ife: fix a potential deadlock")
> Signed-off-by: Vlad Buslov <vla...@mellanox.com>

This patch should be applied to -net rather than -net-next.

Acked-by: Cong Wang <xiyou.wangc...@gmail.com>

One nit below.

>  static int add_metainfo(struct tcf_ife_info *ife, u32 metaid, void *metaval,
>                         int len, bool exists)
>  {
> @@ -349,7 +364,8 @@ static int use_all_metadata(struct tcf_ife_info *ife, 
> bool exists)
>
>         read_lock(&ife_mod_lock);
>         list_for_each_entry(o, &ifeoplist, list) {
> -               rc = __add_metainfo(o, ife, o->metaid, NULL, 0, true, exists);
> +               rc = add_metainfo_and_get_ops(o, ife, o->metaid, NULL, 0, 
> true,
> +                                             exists);

Nit: you can fold constants into this helper as it only has one caller.

Reply via email to