As mentioned by Alexei last week in Budapest, it is a bit weird to take a spinlock in order to drop a packet in a tc filter...
Lets add percpu infra for tc actions and use it for gact & mirred. Before changes, my host with 8 RX queues was handling 5 Mpps with gact, and more than 11 Mpps after. Mirred change is not yet visible if ifb+qdisc is used, as ifb is not yet multi queue enabled, but is a step forward. Signed-off-by: Eric Dumazet <eduma...@google.com> Cc: Alexei Starovoitov <a...@plumgrid.com> Cc: Jamal Hadi Salim <j...@mojatatu.com> Cc: John Fastabend <john.fastab...@gmail.com> Eric Dumazet (7): net: sched: extend percpu stats helpers net: sched: add percpu stats to actions net_sched: act_gact: make tcfg_pval non zero net_sched: act_gact: use a separate packet counters for gact_determ() net_sched: act_gact: read tcfg_ptype once net_sched: act_gact: remove spinlock in fast path net_sched: act_mirred: remove spinlock in fast path include/net/act_api.h | 15 ++++++++++- include/net/sch_generic.h | 31 ++++++++++++++-------- include/net/tc_act/tc_gact.h | 7 ++--- include/net/tc_act/tc_mirred.h | 2 +- net/core/dev.c | 4 +-- net/sched/act_api.c | 44 ++++++++++++++++++++++++-------- net/sched/act_bpf.c | 2 +- net/sched/act_connmark.c | 3 ++- net/sched/act_csum.c | 3 ++- net/sched/act_gact.c | 44 ++++++++++++++++++-------------- net/sched/act_ipt.c | 2 +- net/sched/act_mirred.c | 58 ++++++++++++++++++++++-------------------- net/sched/act_nat.c | 3 ++- net/sched/act_pedit.c | 3 ++- net/sched/act_simple.c | 3 ++- net/sched/act_skbedit.c | 3 ++- net/sched/act_vlan.c | 3 ++- 17 files changed, 148 insertions(+), 82 deletions(-) -- 2.4.3.573.g4eafbef -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
