]> git.baikalelectronics.ru Git - kernel.git/commit
net: core: set qdisc pkt len before tc_classify
authorFlorian Westphal <fw@strlen.de>
Wed, 13 May 2015 22:36:28 +0000 (00:36 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 15 May 2015 02:44:40 +0000 (22:44 -0400)
commit71696be8cd73555e9b3a7652237206ef49276f71
treecd881d3e91edf3c84ac2e8766e9aef242120b2d6
parent8c5a26522937319144fce5fc3f4afb7605a28ad2
net: core: set qdisc pkt len before tc_classify

commit a7d7304048a2385499 ("net: sched: further simplify handle_ing")
removed the call to qdisc_enqueue_root().

However, after this removal we no longer set qdisc pkt length.
This breaks traffic policing on ingress.

This is the minimum fix: set qdisc pkt length before tc_classify.

Only setting the length does remove support for 'stab' on ingress, but
as Alexei pointed out:
 "Though it was allowed to add qdisc_size_table to ingress, it's useless.
  Nothing takes advantage of recomputed qdisc_pkt_len".

Jamal suggested to use qdisc_pkt_len_init(), but as Eric mentioned that
would result in qdisc_pkt_len_init to no longer get inlined due to the
additional 2nd call site.

ingress policing is rare and GRO doesn't really work that well with police
on ingress, as we see packets > mtu and drop skbs that  -- without
aggregation -- would still have fitted the policier budget.
Thus to have reliable/smooth ingress policing GRO has to be turned off.

Cc: Alexei Starovoitov <alexei.starovoitov@gmail.com>
Cc: Eric Dumazet <eric.dumazet@gmail.com>
Cc: Jamal Hadi Salim <jhs@mojatatu.com>
Fixes: a7d7304048a2 ("net: sched: further simplify handle_ing")
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: Florian Westphal <fw@strlen.de>
Acked-by: Eric Dumazet <edumazet@google.com>
Acked-by: Alexei Starovoitov <ast@plumgrid.com>
Acked-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c