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


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