From: Thomas Graf <tgraf@suug.ch>
To: "David S. Miller" <davem@davemloft.net>
Cc: netdev@oss.sgi.com, hadi@cyberus.ca
Subject: [RESEND 1/6] PKT_SCHED: Add generic classifier routines
Date: Fri, 29 Oct 2004 12:23:27 +0200 [thread overview]
Message-ID: <20041029102327.GG12289@postel.suug.ch> (raw)
In-Reply-To: <20041029002229.GZ12289@postel.suug.ch>
CONFIG_NET_CLS_IND was dependent on CONFIG_NET_CLS_ACT in cls_fw.c which
is not correct according to Kconfig and wouldn't make sense. This is a
revised patch which has this dependency removed.
Adds generic routines used by classifier to:
- bind/unbind to classes
- configure action/police/indev
- dump action/police
Signed-off-by: Thomas Graf <tgraf@suug.ch>
--- linux-2.6.10-rc1-bk7.orig/include/net/pkt_cls.h 2004-10-28 22:03:20.000000000 +0200
+++ linux-2.6.10-rc1-bk7/include/net/pkt_cls.h 2004-10-29 12:11:54.000000000 +0200
@@ -3,6 +3,7 @@
#include <linux/pkt_cls.h>
#include <net/sch_generic.h>
+#include <net/act_api.h>
/* Basic packet classifier frontend definitions. */
@@ -41,4 +42,189 @@
return old_cl;
}
+static inline void
+tcf_bind_filter(struct tcf_proto *tp, struct tcf_result *r, unsigned long base)
+{
+ unsigned long cl;
+
+ cl = tp->q->ops->cl_ops->bind_tcf(tp->q, base, r->classid);
+ cl = cls_set_class(tp, &r->class, cl);
+ if (cl)
+ tp->q->ops->cl_ops->unbind_tcf(tp->q, cl);
+}
+
+static inline void
+tcf_unbind_filter(struct tcf_proto *tp, struct tcf_result *r)
+{
+ unsigned long cl;
+
+ if ((cl = __cls_set_class(&r->class, 0)) != 0)
+ tp->q->ops->cl_ops->unbind_tcf(tp->q, cl);
+}
+
+#ifdef CONFIG_NET_CLS_ACT
+static inline int
+tcf_change_act_police(struct tcf_proto *tp, struct tc_action **action,
+ struct rtattr *act_police_tlv, struct rtattr *rate_tlv)
+{
+ int ret;
+ struct tc_action *act;
+
+ act = kmalloc(sizeof(*act), GFP_KERNEL);
+ if (NULL == act)
+ return -ENOMEM;
+ memset(act, 0, sizeof(*act));
+
+ ret = tcf_action_init_1(act_police_tlv, rate_tlv, act, "police",
+ TCA_ACT_NOREPLACE, TCA_ACT_BIND);
+ if (ret < 0) {
+ tcf_action_destroy(act, TCA_ACT_UNBIND);
+ return ret;
+ }
+
+ act->type = TCA_OLD_COMPAT;
+
+ if (*action) {
+ tcf_tree_lock(tp);
+ act = xchg(action, act);
+ tcf_tree_unlock(tp);
+
+ tcf_action_destroy(act, TCA_ACT_UNBIND);
+ } else
+ *action = act;
+
+ return 0;
+}
+
+static inline int
+tcf_change_act(struct tcf_proto *tp, struct tc_action **action,
+ struct rtattr *act_tlv, struct rtattr *rate_tlv)
+{
+ int ret;
+ struct tc_action *act;
+
+ act = kmalloc(sizeof(*act), GFP_KERNEL);
+ if (NULL == act)
+ return -ENOMEM;
+ memset(act, 0, sizeof(*act));
+
+ ret = tcf_action_init(act_tlv, rate_tlv, act, NULL,
+ TCA_ACT_NOREPLACE, TCA_ACT_BIND);
+ if (ret < 0) {
+ tcf_action_destroy(act, TCA_ACT_UNBIND);
+ return ret;
+ }
+
+ if (*action) {
+ tcf_tree_lock(tp);
+ act = xchg(action, act);
+ tcf_tree_unlock(tp);
+
+ tcf_action_destroy(act, TCA_ACT_UNBIND);
+ } else
+ *action = act;
+
+ return 0;
+}
+
+static inline int
+tcf_dump_act(struct sk_buff *skb, struct tc_action *action,
+ int act_type, int compat_type)
+{
+ /*
+ * again for backward compatible mode - we want
+ * to work with both old and new modes of entering
+ * tc data even if iproute2 was newer - jhs
+ */
+ if (action) {
+ struct rtattr * p_rta = (struct rtattr*) skb->tail;
+
+ if (action->type != TCA_OLD_COMPAT) {
+ RTA_PUT(skb, act_type, 0, NULL);
+ if (tcf_action_dump(skb, action, 0, 0) < 0)
+ goto rtattr_failure;
+ } else {
+ RTA_PUT(skb, compat_type, 0, NULL);
+ if (tcf_action_dump_old(skb, action, 0, 0) < 0)
+ goto rtattr_failure;
+ }
+
+ p_rta->rta_len = skb->tail - (u8*)p_rta;
+ }
+ return 0;
+
+rtattr_failure:
+ return -1;
+}
+#endif /* CONFIG_NET_CLS_ACT */
+
+#ifdef CONFIG_NET_CLS_IND
+static inline int
+tcf_change_indev(struct tcf_proto *tp, char *indev, struct rtattr *indev_tlv)
+{
+ if (RTA_PAYLOAD(indev_tlv) >= IFNAMSIZ) {
+ printk("cls: bad indev name %s\n", (char *) RTA_DATA(indev_tlv));
+ return -EINVAL;
+ }
+
+ memset(indev, 0, IFNAMSIZ);
+ sprintf(indev, "%s", (char *) RTA_DATA(indev_tlv));
+
+ return 0;
+}
+
+static inline int
+tcf_match_indev(struct sk_buff *skb, char *indev)
+{
+ if (0 != indev[0]) {
+ if (NULL == skb->input_dev)
+ return 0;
+ else if (0 != strcmp(indev, skb->input_dev->name))
+ return 0;
+ }
+
+ return 1;
+}
+#endif /* CONFIG_NET_CLS_IND */
+
+#ifdef CONFIG_NET_CLS_POLICE
+static inline int
+tcf_change_police(struct tcf_proto *tp, struct tcf_police **police,
+ struct rtattr *police_tlv, struct rtattr *rate_tlv)
+{
+ struct tcf_police *p = tcf_police_locate(police_tlv, rate_tlv);
+
+ if (*police) {
+ tcf_tree_lock(tp);
+ p = xchg(police, p);
+ tcf_tree_unlock(tp);
+
+ tcf_police_release(p, TCA_ACT_UNBIND);
+ } else
+ *police = p;
+
+ return 0;
+}
+
+static inline int
+tcf_dump_police(struct sk_buff *skb, struct tcf_police *police,
+ int police_type)
+{
+ if (police) {
+ struct rtattr * p_rta = (struct rtattr*) skb->tail;
+
+ RTA_PUT(skb, police_type, 0, NULL);
+
+ if (tcf_police_dump(skb, police) < 0)
+ goto rtattr_failure;
+
+ p_rta->rta_len = skb->tail - (u8*)p_rta;
+ }
+ return 0;
+
+rtattr_failure:
+ return -1;
+}
+#endif /* CONFIG_NET_CLS_POLICE */
+
#endif
next prev parent reply other threads:[~2004-10-29 10:23 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-10-29 0:21 [PATCHSET 0/6] PKT_SCHED: Generic classifier routines / cls_fw cleanup Thomas Graf
2004-10-29 0:22 ` [PATCH 1/6] PKT_SCHED: Add generic classifier routines Thomas Graf
2004-10-29 10:23 ` Thomas Graf [this message]
2004-10-29 0:23 ` [PATCH 2/6] cls_fw: Cleanup fw_classify Thomas Graf
2004-10-29 0:24 ` [PATCH 3/6] cls_fw: Use generic routines to configure action/policer Thomas Graf
2004-10-29 0:24 ` [PATCH 4/6] cls_fw: Use generic routines to dump action/policer Thomas Graf
2004-10-29 0:25 ` [PATCH 5/6] cls_fw: Whitespace/ifdef fixes Thomas Graf
2004-10-29 0:26 ` [PATCH 6/6] PKT_SCHED: break is not enough to stop walking Thomas Graf
2004-10-29 10:26 ` [PATCH 7/6] cls_fw: CONFIG_NET_CLS_IND is not dependant on CONFIG_NET_CLS_ACT Thomas Graf
2004-10-29 11:39 ` jamal
2004-10-29 11:53 ` Thomas Graf
2004-10-29 12:35 ` jamal
2004-10-29 12:53 ` Thomas Graf
2004-10-29 13:12 ` jamal
2004-10-29 14:24 ` Thomas Graf
2004-10-29 14:53 ` jamal
2004-11-02 0:39 ` David S. Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20041029102327.GG12289@postel.suug.ch \
--to=tgraf@suug.ch \
--cc=davem@davemloft.net \
--cc=hadi@cyberus.ca \
--cc=netdev@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).