Re: [NET_SCHED 00/04]: External SFQ classifiers/flow classifier
Patrick McHardy wrote: > You're missing protocol, handle etc. Try something like this: > > tc filter add dev eth0 protocol ip pref 1 parent 1: handle 1 \ > > flow hash keys dst divisor 1024 Thanks, the kernel accepts that. I guess I understand tc filter usage less than I thought I did Time to teach myself better. -Corey -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [NET_SCHED 00/04]: External SFQ classifiers/flow classifier
Corey Hickey wrote: Patrick McHardy wrote: These patches add support for external classifiers to SFQ and add a new "flow" classifier, which can do hashing based on user-specified keys or deterministic mapping of keys to classes. Additionally there is a patch to make the SFQ queues visisble as classes to verify that the hash is indeed doing something useful and a patch to consifiy struct tcf_ext_map, which I had queued in the same tree. Excellent! I'm glad this is applied. I'm having trouble figuring out how it works, though. As a test, I'm trying to set up SFQ equivalent to ESFQ's "hash dst". Here's what I do, and this is what I get: # ./tc qdisc add dev eth0 root handle 1: sfq # ./tc filter add dev eth0 parent 1: flow hash keys dst RTNETLINK answers: Invalid argument We have an error talking to the kernel I've tried a few different keys with the same results. I don't know what I'm doing wrong, or even where to start figuring it out. Can you point me in the right direction? You're missing protocol, handle etc. Try something like this: tc filter add dev eth0 protocol ip pref 1 parent 1: handle 1 \ flow hash keys dst divisor 1024 -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [NET_SCHED 00/04]: External SFQ classifiers/flow classifier
Patrick McHardy wrote: > These patches add support for external classifiers to SFQ and add a > new "flow" classifier, which can do hashing based on user-specified > keys or deterministic mapping of keys to classes. Additionally there > is a patch to make the SFQ queues visisble as classes to verify that > the hash is indeed doing something useful and a patch to consifiy > struct tcf_ext_map, which I had queued in the same tree. Excellent! I'm glad this is applied. I'm having trouble figuring out how it works, though. As a test, I'm trying to set up SFQ equivalent to ESFQ's "hash dst". Here's what I do, and this is what I get: # ./tc qdisc add dev eth0 root handle 1: sfq # ./tc filter add dev eth0 parent 1: flow hash keys dst RTNETLINK answers: Invalid argument We have an error talking to the kernel I've tried a few different keys with the same results. I don't know what I'm doing wrong, or even where to start figuring it out. Can you point me in the right direction? Here are some details that may be pertinent: - current net-2.6 git (I double-checked) - CONFIG_NET_CLS_FLOW=y - current iproute2 git - running on amd64 user-mode Linux Thanks, Corey -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [NET_SCHED 00/04]: External SFQ classifiers/flow classifier
From: Patrick McHardy <[EMAIL PROTECTED]> Date: Thu, 31 Jan 2008 18:58:02 +0100 (MET) > These patches add support for external classifiers to SFQ and add a > new "flow" classifier, which can do hashing based on user-specified > keys or deterministic mapping of keys to classes. Additionally there > is a patch to make the SFQ queues visisble as classes to verify that > the hash is indeed doing something useful and a patch to consifiy > struct tcf_ext_map, which I had queued in the same tree. > > Please apply, thanks. Applied, thanks Patrick. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
[NET_SCHED 00/04]: External SFQ classifiers/flow classifier
These patches add support for external classifiers to SFQ and add a new "flow" classifier, which can do hashing based on user-specified keys or deterministic mapping of keys to classes. Additionally there is a patch to make the SFQ queues visisble as classes to verify that the hash is indeed doing something useful and a patch to consifiy struct tcf_ext_map, which I had queued in the same tree. Please apply, thanks. include/linux/pkt_cls.h | 50 include/linux/pkt_sched.h |5 + include/net/pkt_cls.h |6 +- net/sched/Kconfig | 11 + net/sched/Makefile|1 + net/sched/cls_api.c |6 +- net/sched/cls_basic.c |2 +- net/sched/cls_flow.c | 660 + net/sched/cls_fw.c|2 +- net/sched/cls_route.c |2 +- net/sched/cls_tcindex.c |2 +- net/sched/cls_u32.c |2 +- net/sched/sch_sfq.c | 134 +- 13 files changed, 868 insertions(+), 15 deletions(-) create mode 100644 net/sched/cls_flow.c Patrick McHardy (4): [NET_SCHED]: Constify struct tcf_ext_map [NET_SCHED]: sch_sfq: add support for external classifiers [NET_SCHED]: sch_sfq: make internal queues visible as classes [NET_SCHED]: Add flow classifier -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html