Re: [NET_SCHED 00/04]: External SFQ classifiers/flow classifier

2008-02-04 Thread Patrick McHardy

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

2008-02-04 Thread Corey Hickey
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

2008-02-02 Thread Corey Hickey
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


[NET_SCHED 00/04]: External SFQ classifiers/flow classifier

2008-01-31 Thread Patrick McHardy
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


Re: [NET_SCHED 00/04]: External SFQ classifiers/flow classifier

2008-01-31 Thread David Miller
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