diff options
| author | Vlad Buslov <vladbu@mellanox.com> | 2018-07-05 16:24:28 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2018-07-08 05:42:29 +0200 |
| commit | b409074e6693bcdaa7abbee2a035f22a9eabda53 (patch) | |
| tree | f531c8c864cbffaa0c2258d2dada07b1fc03d33b /net/sched/act_gact.c | |
| parent | net: sched: implement action API that deletes action by index (diff) | |
| download | linux-b409074e6693bcdaa7abbee2a035f22a9eabda53.tar.xz linux-b409074e6693bcdaa7abbee2a035f22a9eabda53.zip | |
net: sched: add 'delete' function to action ops
Extend action ops with 'delete' function. Each action type to implements
its own delete function that doesn't depend on rtnl lock.
Implement delete function that is required to delete actions without
holding rtnl lock. Use action API function that atomically deletes action
only if it is still in action idr. This implementation prevents concurrent
threads from deleting same action twice.
Reviewed-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: Vlad Buslov <vladbu@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/act_gact.c')
| -rw-r--r-- | net/sched/act_gact.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/net/sched/act_gact.c b/net/sched/act_gact.c index b4dfb2b4addc..a431a711f0dd 100644 --- a/net/sched/act_gact.c +++ b/net/sched/act_gact.c @@ -231,6 +231,13 @@ static size_t tcf_gact_get_fill_size(const struct tc_action *act) return sz; } +static int tcf_gact_delete(struct net *net, u32 index) +{ + struct tc_action_net *tn = net_generic(net, gact_net_id); + + return tcf_idr_delete_index(tn, index); +} + static struct tc_action_ops act_gact_ops = { .kind = "gact", .type = TCA_ACT_GACT, @@ -242,6 +249,7 @@ static struct tc_action_ops act_gact_ops = { .walk = tcf_gact_walker, .lookup = tcf_gact_search, .get_fill_size = tcf_gact_get_fill_size, + .delete = tcf_gact_delete, .size = sizeof(struct tcf_gact), }; |