]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
Merge branch 'sched-cls_api-small-cleanup'
authorDavid S. Miller <davem@davemloft.net>
Fri, 10 Feb 2017 16:38:10 +0000 (11:38 -0500)
committerDavid S. Miller <davem@davemloft.net>
Fri, 10 Feb 2017 16:38:10 +0000 (11:38 -0500)
Jiri Pirko says:

====================
sched: cls_api: small cleanup

This patchset makes couple of things in cls_api code a bit nicer and easier
for reader to digest.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>
15 files changed:
include/net/pkt_cls.h
include/net/sch_generic.h
net/sched/cls_api.c
net/sched/sch_api.c
net/sched/sch_atm.c
net/sched/sch_cbq.c
net/sched/sch_choke.c
net/sched/sch_dsmark.c
net/sched/sch_fq_codel.c
net/sched/sch_htb.c
net/sched/sch_ingress.c
net/sched/sch_multiq.c
net/sched/sch_prio.c
net/sched/sch_sfb.c
net/sched/sch_sfq.c

index dabb00af46a0d31e36c45235aa742a124d78df89..71b266cd63d4b0a79a81139d0e4eb890f93d1a09 100644 (file)
@@ -17,6 +17,8 @@ struct tcf_walker {
 int register_tcf_proto_ops(struct tcf_proto_ops *ops);
 int unregister_tcf_proto_ops(struct tcf_proto_ops *ops);
 
+void tcf_destroy_chain(struct tcf_proto __rcu **fl);
+
 static inline unsigned long
 __cls_set_class(unsigned long *clp, unsigned long cl)
 {
index e2f426f6d62f760ff7c4f6deb69947daa39cbe14..aeec4086afb2446dadb1fb8c54ad54a909634380 100644 (file)
@@ -405,8 +405,6 @@ struct Qdisc *qdisc_create_dflt(struct netdev_queue *dev_queue,
                                const struct Qdisc_ops *ops, u32 parentid);
 void __qdisc_calculate_pkt_len(struct sk_buff *skb,
                               const struct qdisc_size_table *stab);
-bool tcf_destroy(struct tcf_proto *tp, bool force);
-void tcf_destroy_chain(struct tcf_proto __rcu **fl);
 int skb_do_redirect(struct sk_buff *);
 
 static inline void skb_reset_tc(struct sk_buff *skb)
index 1ecdf809b5fa8913d56bb4194ac193274d4d625a..abe1fe13ae5405c2402f083156c5b7345acc5ac8 100644 (file)
@@ -19,6 +19,7 @@
 #include <linux/kernel.h>
 #include <linux/string.h>
 #include <linux/errno.h>
+#include <linux/err.h>
 #include <linux/skbuff.h>
 #include <linux/init.h>
 #include <linux/kmod.h>
@@ -38,14 +39,14 @@ static DEFINE_RWLOCK(cls_mod_lock);
 
 /* Find classifier type by string name */
 
-static const struct tcf_proto_ops *tcf_proto_lookup_ops(struct nlattr *kind)
+static const struct tcf_proto_ops *tcf_proto_lookup_ops(const char *kind)
 {
        const struct tcf_proto_ops *t, *res = NULL;
 
        if (kind) {
                read_lock(&cls_mod_lock);
                list_for_each_entry(t, &tcf_proto_base, head) {
-                       if (nla_strcmp(kind, t->kind) == 0) {
+                       if (strcmp(kind, t->kind) == 0) {
                                if (try_module_get(t->owner))
                                        res = t;
                                break;
@@ -127,6 +128,77 @@ static inline u32 tcf_auto_prio(struct tcf_proto *tp)
        return first;
 }
 
+static struct tcf_proto *tcf_proto_create(const char *kind, u32 protocol,
+                                         u32 prio, u32 parent, struct Qdisc *q)
+{
+       struct tcf_proto *tp;
+       int err;
+
+       tp = kzalloc(sizeof(*tp), GFP_KERNEL);
+       if (!tp)
+               return ERR_PTR(-ENOBUFS);
+
+       err = -ENOENT;
+       tp->ops = tcf_proto_lookup_ops(kind);
+       if (!tp->ops) {
+#ifdef CONFIG_MODULES
+               rtnl_unlock();
+               request_module("cls_%s", kind);
+               rtnl_lock();
+               tp->ops = tcf_proto_lookup_ops(kind);
+               /* We dropped the RTNL semaphore in order to perform
+                * the module load. So, even if we succeeded in loading
+                * the module we have to replay the request. We indicate
+                * this using -EAGAIN.
+                */
+               if (tp->ops) {
+                       module_put(tp->ops->owner);
+                       err = -EAGAIN;
+               } else {
+                       err = -ENOENT;
+               }
+               goto errout;
+#endif
+       }
+       tp->classify = tp->ops->classify;
+       tp->protocol = protocol;
+       tp->prio = prio;
+       tp->classid = parent;
+       tp->q = q;
+
+       err = tp->ops->init(tp);
+       if (err) {
+               module_put(tp->ops->owner);
+               goto errout;
+       }
+       return tp;
+
+errout:
+       kfree(tp);
+       return ERR_PTR(err);
+}
+
+static bool tcf_proto_destroy(struct tcf_proto *tp, bool force)
+{
+       if (tp->ops->destroy(tp, force)) {
+               module_put(tp->ops->owner);
+               kfree_rcu(tp, rcu);
+               return true;
+       }
+       return false;
+}
+
+void tcf_destroy_chain(struct tcf_proto __rcu **fl)
+{
+       struct tcf_proto *tp;
+
+       while ((tp = rtnl_dereference(*fl)) != NULL) {
+               RCU_INIT_POINTER(*fl, tp->next);
+               tcf_proto_destroy(tp, true);
+       }
+}
+EXPORT_SYMBOL(tcf_destroy_chain);
+
 /* Add/change/delete/get a filter node */
 
 static int tc_ctl_tfilter(struct sk_buff *skb, struct nlmsghdr *n)
@@ -142,8 +214,8 @@ static int tc_ctl_tfilter(struct sk_buff *skb, struct nlmsghdr *n)
        struct Qdisc  *q;
        struct tcf_proto __rcu **back;
        struct tcf_proto __rcu **chain;
+       struct tcf_proto *next;
        struct tcf_proto *tp;
-       const struct tcf_proto_ops *tp_ops;
        const struct Qdisc_class_ops *cops;
        unsigned long cl;
        unsigned long fh;
@@ -222,9 +294,10 @@ replay:
 
        /* And the last stroke */
        chain = cops->tcf_chain(q, cl);
-       err = -EINVAL;
-       if (chain == NULL)
+       if (chain == NULL) {
+               err = -EINVAL;
                goto errout;
+       }
        if (n->nlmsg_type == RTM_DELTFILTER && prio == 0) {
                tfilter_notify_chain(net, skb, n, chain, RTM_DELTFILTER);
                tcf_destroy_chain(chain);
@@ -239,10 +312,13 @@ replay:
                if (tp->prio >= prio) {
                        if (tp->prio == prio) {
                                if (!nprio ||
-                                   (tp->protocol != protocol && protocol))
+                                   (tp->protocol != protocol && protocol)) {
+                                       err = -EINVAL;
                                        goto errout;
-                       } else
+                               }
+                       } else {
                                tp = NULL;
+                       }
                        break;
                }
        }
@@ -250,110 +326,69 @@ replay:
        if (tp == NULL) {
                /* Proto-tcf does not exist, create new one */
 
-               if (tca[TCA_KIND] == NULL || !protocol)
+               if (tca[TCA_KIND] == NULL || !protocol) {
+                       err = -EINVAL;
                        goto errout;
+               }
 
-               err = -ENOENT;
                if (n->nlmsg_type != RTM_NEWTFILTER ||
-                   !(n->nlmsg_flags & NLM_F_CREATE))
+                   !(n->nlmsg_flags & NLM_F_CREATE)) {
+                       err = -ENOENT;
                        goto errout;
+               }
 
+               if (!nprio)
+                       nprio = TC_H_MAJ(tcf_auto_prio(rtnl_dereference(*back)));
 
-               /* Create new proto tcf */
-
-               err = -ENOBUFS;
-               tp = kzalloc(sizeof(*tp), GFP_KERNEL);
-               if (tp == NULL)
-                       goto errout;
-               err = -ENOENT;
-               tp_ops = tcf_proto_lookup_ops(tca[TCA_KIND]);
-               if (tp_ops == NULL) {
-#ifdef CONFIG_MODULES
-                       struct nlattr *kind = tca[TCA_KIND];
-                       char name[IFNAMSIZ];
-
-                       if (kind != NULL &&
-                           nla_strlcpy(name, kind, IFNAMSIZ) < IFNAMSIZ) {
-                               rtnl_unlock();
-                               request_module("cls_%s", name);
-                               rtnl_lock();
-                               tp_ops = tcf_proto_lookup_ops(kind);
-                               /* We dropped the RTNL semaphore in order to
-                                * perform the module load.  So, even if we
-                                * succeeded in loading the module we have to
-                                * replay the request.  We indicate this using
-                                * -EAGAIN.
-                                */
-                               if (tp_ops != NULL) {
-                                       module_put(tp_ops->owner);
-                                       err = -EAGAIN;
-                               }
-                       }
-#endif
-                       kfree(tp);
+               tp = tcf_proto_create(nla_data(tca[TCA_KIND]),
+                                     protocol, nprio, parent, q);
+               if (IS_ERR(tp)) {
+                       err = PTR_ERR(tp);
                        goto errout;
                }
-               tp->ops = tp_ops;
-               tp->protocol = protocol;
-               tp->prio = nprio ? :
-                              TC_H_MAJ(tcf_auto_prio(rtnl_dereference(*back)));
-               tp->q = q;
-               tp->classify = tp_ops->classify;
-               tp->classid = parent;
-
-               err = tp_ops->init(tp);
-               if (err != 0) {
-                       module_put(tp_ops->owner);
-                       kfree(tp);
-                       goto errout;
-               }
-
                tp_created = 1;
-
-       } else if (tca[TCA_KIND] && nla_strcmp(tca[TCA_KIND], tp->ops->kind))
+       } else if (tca[TCA_KIND] && nla_strcmp(tca[TCA_KIND], tp->ops->kind)) {
+               err = -EINVAL;
                goto errout;
+       }
 
        fh = tp->ops->get(tp, t->tcm_handle);
 
        if (fh == 0) {
                if (n->nlmsg_type == RTM_DELTFILTER && t->tcm_handle == 0) {
-                       struct tcf_proto *next = rtnl_dereference(tp->next);
-
+                       next = rtnl_dereference(tp->next);
                        RCU_INIT_POINTER(*back, next);
-
                        tfilter_notify(net, skb, n, tp, fh,
                                       RTM_DELTFILTER, false);
-                       tcf_destroy(tp, true);
+                       tcf_proto_destroy(tp, true);
                        err = 0;
                        goto errout;
                }
 
-               err = -ENOENT;
                if (n->nlmsg_type != RTM_NEWTFILTER ||
-                   !(n->nlmsg_flags & NLM_F_CREATE))
+                   !(n->nlmsg_flags & NLM_F_CREATE)) {
+                       err = -ENOENT;
                        goto errout;
+               }
        } else {
                switch (n->nlmsg_type) {
                case RTM_NEWTFILTER:
-                       err = -EEXIST;
                        if (n->nlmsg_flags & NLM_F_EXCL) {
                                if (tp_created)
-                                       tcf_destroy(tp, true);
+                                       tcf_proto_destroy(tp, true);
+                               err = -EEXIST;
                                goto errout;
                        }
                        break;
                case RTM_DELTFILTER:
                        err = tp->ops->delete(tp, fh);
-                       if (err == 0) {
-                               struct tcf_proto *next = rtnl_dereference(tp->next);
-
-                               tfilter_notify(net, skb, n, tp,
-                                              t->tcm_handle,
-                                              RTM_DELTFILTER, false);
-                               if (tcf_destroy(tp, false))
-                                       RCU_INIT_POINTER(*back, next);
-                       }
-                       goto errout;
+                       if (err)
+                               goto errout;
+                       next = rtnl_dereference(tp->next);
+                       tfilter_notify(net, skb, n, tp, t->tcm_handle,
+                                      RTM_DELTFILTER, false);
+                       if (tcf_proto_destroy(tp, false))
+                               RCU_INIT_POINTER(*back, next);
                case RTM_GETTFILTER:
                        err = tfilter_notify(net, skb, n, tp, fh,
                                             RTM_NEWTFILTER, true);
@@ -374,7 +409,7 @@ replay:
                tfilter_notify(net, skb, n, tp, fh, RTM_NEWTFILTER, false);
        } else {
                if (tp_created)
-                       tcf_destroy(tp, true);
+                       tcf_proto_destroy(tp, true);
        }
 
 errout:
index ef53ede11590924f51bca17f79fb8ad9697366be..adeabaec0d0b8bd3115e8d8db756460227142c60 100644 (file)
@@ -1900,28 +1900,6 @@ reset:
 }
 EXPORT_SYMBOL(tc_classify);
 
-bool tcf_destroy(struct tcf_proto *tp, bool force)
-{
-       if (tp->ops->destroy(tp, force)) {
-               module_put(tp->ops->owner);
-               kfree_rcu(tp, rcu);
-               return true;
-       }
-
-       return false;
-}
-
-void tcf_destroy_chain(struct tcf_proto __rcu **fl)
-{
-       struct tcf_proto *tp;
-
-       while ((tp = rtnl_dereference(*fl)) != NULL) {
-               RCU_INIT_POINTER(*fl, tp->next);
-               tcf_destroy(tp, true);
-       }
-}
-EXPORT_SYMBOL(tcf_destroy_chain);
-
 #ifdef CONFIG_PROC_FS
 static int psched_show(struct seq_file *seq, void *v)
 {
index 481e4f12aeb4c05e17f6985764cd901c9fa59331..2209c2ddacbfb0b1f222cb593477891d7124d9e3 100644 (file)
@@ -15,6 +15,7 @@
 #include <linux/file.h>                /* for fput */
 #include <net/netlink.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 
 /*
  * The ATM queuing discipline provides a framework for invoking classifiers
index f1207582cbf3005f08af70275d6c1b217f60f26a..d6ca18dc04c3e9e72efedd44088e95118a06b711 100644 (file)
@@ -19,6 +19,7 @@
 #include <linux/skbuff.h>
 #include <net/netlink.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 
 
 /*     Class-Based Queueing (CBQ) algorithm.
index 3b6d5bd691015f0dbfa285218e2453a8587b8043..3b86a97bc67c3e953cb181eddcb5c0c16bf3b27f 100644 (file)
@@ -16,6 +16,7 @@
 #include <linux/skbuff.h>
 #include <linux/vmalloc.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 #include <net/inet_ecn.h>
 #include <net/red.h>
 #include <net/flow_dissector.h>
index 1308bbf460f7ca7e496fe9d9c48da7eb2b0d282d..802ac7c2e5e87eed1341ba4c09d3e5d70bc75876 100644 (file)
@@ -13,6 +13,7 @@
 #include <linux/rtnetlink.h>
 #include <linux/bitops.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 #include <net/dsfield.h>
 #include <net/inet_ecn.h>
 #include <asm/byteorder.h>
index 2f50e4c72fb406d4aa67b868ceb37ae72fa60bf6..9f3a884d15903fd9012c01b5eee802e02f9f709e 100644 (file)
@@ -23,6 +23,7 @@
 #include <linux/vmalloc.h>
 #include <net/netlink.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 #include <net/codel.h>
 #include <net/codel_impl.h>
 #include <net/codel_qdisc.h>
index 760f39e7caeeb91b6c34d561f64f1ed97c884a32..4cd5fb134bc9e2dbcdd61b51fb951f94301ed54c 100644 (file)
@@ -40,6 +40,7 @@
 #include <net/netlink.h>
 #include <net/sch_generic.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 
 /* HTB algorithm.
     Author: devik@cdi.cz
index 8fe6999b642ac3c039f40621f4ca123d129718c4..3bab5f66c39291fb4f20888dbdb779dc7edc2776 100644 (file)
@@ -16,6 +16,7 @@
 
 #include <net/netlink.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 
 static struct Qdisc *ingress_leaf(struct Qdisc *sch, unsigned long arg)
 {
index 9ffbb025b37e70bc526f54735a9b16573efb0ff4..e7839a0d0eaa52572f675fdb1dfc590c2a70ac76 100644 (file)
@@ -25,7 +25,7 @@
 #include <linux/skbuff.h>
 #include <net/netlink.h>
 #include <net/pkt_sched.h>
-
+#include <net/pkt_cls.h>
 
 struct multiq_sched_data {
        u16 bands;
index 8f575899adfa61cb985c0c645d265e1c9381c9d9..d4d7db267b6edfa56582ca4a588590e0ded9fe66 100644 (file)
@@ -20,7 +20,7 @@
 #include <linux/skbuff.h>
 #include <net/netlink.h>
 #include <net/pkt_sched.h>
-
+#include <net/pkt_cls.h>
 
 struct prio_sched_data {
        int bands;
index 20a350bd1b1dc8283c7e352ce0a953ce0eea2544..fe6963d2151956c508b510edec680b89201173ce 100644 (file)
@@ -25,6 +25,7 @@
 #include <linux/jhash.h>
 #include <net/ip.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 #include <net/inet_ecn.h>
 
 /*
index 7f195ed4d568c14c618c59f8d746cdd29f8dae21..83d06e251b93cda13938cc00d75ff7c45d7785ad 100644 (file)
@@ -23,6 +23,7 @@
 #include <linux/vmalloc.h>
 #include <net/netlink.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 #include <net/red.h>