net_sched: cls_bpf: remove unnecessary iteration and use passed arg
Signed-off-by: Jiri Pirko <jiri@resnulli.us> Acked-by: Jamal Hadi Salim <jhs@mojatatu.com> Acked-by: Daniel Borkmann <dborkman@redhat.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
e4386456ae
commit
472f583701
|
@ -109,19 +109,12 @@ static void __cls_bpf_delete_prog(struct rcu_head *rcu)
|
||||||
|
|
||||||
static int cls_bpf_delete(struct tcf_proto *tp, unsigned long arg)
|
static int cls_bpf_delete(struct tcf_proto *tp, unsigned long arg)
|
||||||
{
|
{
|
||||||
struct cls_bpf_head *head = rtnl_dereference(tp->root);
|
struct cls_bpf_prog *prog = (struct cls_bpf_prog *) arg;
|
||||||
struct cls_bpf_prog *prog, *todel = (struct cls_bpf_prog *) arg;
|
|
||||||
|
|
||||||
list_for_each_entry(prog, &head->plist, link) {
|
list_del_rcu(&prog->link);
|
||||||
if (prog == todel) {
|
tcf_unbind_filter(tp, &prog->res);
|
||||||
list_del_rcu(&prog->link);
|
call_rcu(&prog->rcu, __cls_bpf_delete_prog);
|
||||||
tcf_unbind_filter(tp, &prog->res);
|
return 0;
|
||||||
call_rcu(&prog->rcu, __cls_bpf_delete_prog);
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return -ENOENT;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void cls_bpf_destroy(struct tcf_proto *tp)
|
static void cls_bpf_destroy(struct tcf_proto *tp)
|
||||||
|
|
Loading…
Reference in New Issue