net: sched: flower: don't take rtnl lock for cls hw offloads API
authorVlad Buslov <vladbu@mellanox.com>
Mon, 26 Aug 2019 13:45:06 +0000 (16:45 +0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 26 Aug 2019 21:17:43 +0000 (14:17 -0700)
Don't manually take rtnl lock in flower classifier before calling cls
hardware offloads API. Instead, pass rtnl lock status via 'rtnl_held'
parameter.

Signed-off-by: Vlad Buslov <vladbu@mellanox.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/cls_flower.c

index 2852fe6f50d2ac3fbfc27ea099d9b9a4ee6b33f1..74221e3351c36a577379edd5d31671041a8a6f39 100644 (file)
@@ -412,18 +412,13 @@ static void fl_hw_destroy_filter(struct tcf_proto *tp, struct cls_fl_filter *f,
        struct tcf_block *block = tp->chain->block;
        struct flow_cls_offload cls_flower = {};
 
-       if (!rtnl_held)
-               rtnl_lock();
-
        tc_cls_common_offload_init(&cls_flower.common, tp, f->flags, extack);
        cls_flower.command = FLOW_CLS_DESTROY;
        cls_flower.cookie = (unsigned long) f;
 
        tc_setup_cb_destroy(block, tp, TC_SETUP_CLSFLOWER, &cls_flower, false,
-                           &f->flags, &f->in_hw_count, true);
+                           &f->flags, &f->in_hw_count, rtnl_held);
 
-       if (!rtnl_held)
-               rtnl_unlock();
 }
 
 static int fl_hw_replace_filter(struct tcf_proto *tp,
@@ -435,14 +430,9 @@ static int fl_hw_replace_filter(struct tcf_proto *tp,
        bool skip_sw = tc_skip_sw(f->flags);
        int err = 0;
 
-       if (!rtnl_held)
-               rtnl_lock();
-
        cls_flower.rule = flow_rule_alloc(tcf_exts_num_actions(&f->exts));
-       if (!cls_flower.rule) {
-               err = -ENOMEM;
-               goto errout;
-       }
+       if (!cls_flower.rule)
+               return -ENOMEM;
 
        tc_cls_common_offload_init(&cls_flower.common, tp, f->flags, extack);
        cls_flower.command = FLOW_CLS_REPLACE;
@@ -453,36 +443,30 @@ static int fl_hw_replace_filter(struct tcf_proto *tp,
        cls_flower.classid = f->res.classid;
 
        err = tc_setup_flow_action(&cls_flower.rule->action, &f->exts,
-                                  true);
+                                  rtnl_held);
        if (err) {
                kfree(cls_flower.rule);
-               if (skip_sw)
+               if (skip_sw) {
                        NL_SET_ERR_MSG_MOD(extack, "Failed to setup flow action");
-               else
-                       err = 0;
-               goto errout;
+                       return err;
+               }
+               return 0;
        }
 
        err = tc_setup_cb_add(block, tp, TC_SETUP_CLSFLOWER, &cls_flower,
-                             skip_sw, &f->flags, &f->in_hw_count, true);
+                             skip_sw, &f->flags, &f->in_hw_count, rtnl_held);
        tc_cleanup_flow_action(&cls_flower.rule->action);
        kfree(cls_flower.rule);
 
        if (err) {
-               fl_hw_destroy_filter(tp, f, true, NULL);
-               goto errout;
-       }
-
-       if (skip_sw && !(f->flags & TCA_CLS_FLAGS_IN_HW)) {
-               err = -EINVAL;
-               goto errout;
+               fl_hw_destroy_filter(tp, f, rtnl_held, NULL);
+               return err;
        }
 
-errout:
-       if (!rtnl_held)
-               rtnl_unlock();
+       if (skip_sw && !(f->flags & TCA_CLS_FLAGS_IN_HW))
+               return -EINVAL;
 
-       return err;
+       return 0;
 }
 
 static void fl_hw_update_stats(struct tcf_proto *tp, struct cls_fl_filter *f,
@@ -491,22 +475,17 @@ static void fl_hw_update_stats(struct tcf_proto *tp, struct cls_fl_filter *f,
        struct tcf_block *block = tp->chain->block;
        struct flow_cls_offload cls_flower = {};
 
-       if (!rtnl_held)
-               rtnl_lock();
-
        tc_cls_common_offload_init(&cls_flower.common, tp, f->flags, NULL);
        cls_flower.command = FLOW_CLS_STATS;
        cls_flower.cookie = (unsigned long) f;
        cls_flower.classid = f->res.classid;
 
-       tc_setup_cb_call(block, TC_SETUP_CLSFLOWER, &cls_flower, false, true);
+       tc_setup_cb_call(block, TC_SETUP_CLSFLOWER, &cls_flower, false,
+                        rtnl_held);
 
        tcf_exts_stats_update(&f->exts, cls_flower.stats.bytes,
                              cls_flower.stats.pkts,
                              cls_flower.stats.lastused);
-
-       if (!rtnl_held)
-               rtnl_unlock();
 }
 
 static void __fl_put(struct cls_fl_filter *f)