diff options
| author |   Vlad Buslov <vladbu@mellanox.com> | 2020-02-17 12:12:12 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-02-17 14:17:02 -0800 |
| commit | b15e7a6e8d31d6abe2d98929d60ad3a0e6ae4de1 (patch) | |
| tree | 3b1986c5151784433053007868f85e4046cd06de /net/sched/cls_matchall.c | |
| parent | net: sched: refactor ct action helpers to require tcf_lock (diff) | |
| download | linux-dev-b15e7a6e8d31d6abe2d98929d60ad3a0e6ae4de1.tar.xz linux-dev-b15e7a6e8d31d6abe2d98929d60ad3a0e6ae4de1.zip | |
net: sched: don't take rtnl lock during flow_action setup
Refactor tc_setup_flow_action() function not to use rtnl lock and remove
'rtnl_held' argument that is no longer needed.
Signed-off-by: Vlad Buslov <vladbu@mellanox.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/cls_matchall.c')
| -rw-r--r-- | net/sched/cls_matchall.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/sched/cls_matchall.c b/net/sched/cls_matchall.c index 610a0b728161..a34b36adb9b7 100644 --- a/net/sched/cls_matchall.c +++ b/net/sched/cls_matchall.c @@ -97,7 +97,7 @@ static int mall_replace_hw_filter(struct tcf_proto *tp, cls_mall.command = TC_CLSMATCHALL_REPLACE; cls_mall.cookie = cookie; - err = tc_setup_flow_action(&cls_mall.rule->action, &head->exts, true); + err = tc_setup_flow_action(&cls_mall.rule->action, &head->exts); if (err) { kfree(cls_mall.rule); mall_destroy_hw_filter(tp, head, cookie, NULL); @@ -302,7 +302,7 @@ static int mall_reoffload(struct tcf_proto *tp, bool add, flow_setup_cb_t *cb, TC_CLSMATCHALL_REPLACE : TC_CLSMATCHALL_DESTROY; cls_mall.cookie = (unsigned long)head; - err = tc_setup_flow_action(&cls_mall.rule->action, &head->exts, true); + err = tc_setup_flow_action(&cls_mall.rule->action, &head->exts); if (err) { kfree(cls_mall.rule); if (add && tc_skip_sw(head->flags)) { |