net_sched: cls_basic: remove unnecessary iteration and use passed arg
authorJiri Pirko <jiri@resnulli.us>
Tue, 2 Dec 2014 17:00:31 +0000 (18:00 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 9 Dec 2014 01:53:40 +0000 (20:53 -0500)
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
Acked-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/cls_basic.c

index cd61280941e5c72d8c22e4dfe18ec517e90cba53..1c122c7e054991b9bc244ce83ed876fc5e6640b5 100644 (file)
@@ -113,18 +113,12 @@ static void basic_destroy(struct tcf_proto *tp)
 
 static int basic_delete(struct tcf_proto *tp, unsigned long arg)
 {
-       struct basic_head *head = rtnl_dereference(tp->root);
-       struct basic_filter *t, *f = (struct basic_filter *) arg;
-
-       list_for_each_entry(t, &head->flist, link)
-               if (t == f) {
-                       list_del_rcu(&t->link);
-                       tcf_unbind_filter(tp, &t->res);
-                       call_rcu(&t->rcu, basic_delete_filter);
-                       return 0;
-               }
+       struct basic_filter *f = (struct basic_filter *) arg;
 
-       return -ENOENT;
+       list_del_rcu(&f->link);
+       tcf_unbind_filter(tp, &f->res);
+       call_rcu(&f->rcu, basic_delete_filter);
+       return 0;
 }
 
 static const struct nla_policy basic_policy[TCA_BASIC_MAX + 1] = {