]> git.baikalelectronics.ru Git - kernel.git/commit
[NET_SCHED] sch_prio.c: remove duplicate call of tc_classify()
authorLucas Nussbaum <lucas.nussbaum@imag.fr>
Fri, 31 Aug 2007 05:35:46 +0000 (22:35 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 31 Aug 2007 05:35:46 +0000 (22:35 -0700)
commit8b6170882da6f1e823a1bffa2e5abf7087caf505
tree2cc33dfd6509c6cac456ae0ccfe386333aaec70a
parentc5b816e657478832f4d28cce969ba3bb88492af4
[NET_SCHED] sch_prio.c: remove duplicate call of tc_classify()

When CONFIG_NET_CLS_ACT is enabled, tc_classify() is called twice in
prio_classify(). This causes "interesting" behaviour: with the setup
below, packets are duplicated, sent twice to ifb0, and then loop in and
out of ifb0.

The patch uses the previously calculated return value in the switch,
which is probably what Patrick had in mind in commit
aad4d97a71c2f81a7a1946ac199ef110eded0dbd -- maybe Patrick can
double-check this?

-- example setup --
ifconfig ifb0 up
tc qdisc add dev ifb0 root netem delay 2s
tc qdisc add dev $ETH root handle 1: prio
tc filter add dev $ETH parent 1: protocol ip prio 10 u32 \
 match ip dst 172.24.110.6/32 flowid 1:1 \
 action mirred egress redirect dev ifb0
ping -c1 172.24.110.6

Signed-off-by: Lucas Nussbaum <lucas.nussbaum@imag.fr>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/sch_prio.c