diff options
| author | Changli Gao <xiaosuo@gmail.com> | 2010-05-02 05:42:16 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-05-02 22:26:57 -0700 |
| commit | dee42870a423ad485129f43cddfe7275479f11d8 (patch) | |
| tree | e30433d3b55ee248eb672765fe6705be160d882c /net/sched | |
| parent | 7ef527377b88ff05fb122a47619ea506c631c914 (diff) | |
| download | op-kernel-dev-dee42870a423ad485129f43cddfe7275479f11d8.zip op-kernel-dev-dee42870a423ad485129f43cddfe7275479f11d8.tar.gz | |
net: fix softnet_stat
Per cpu variable softnet_data.total was shared between IRQ and SoftIRQ context
without any protection. And enqueue_to_backlog should update the netdev_rx_stat
of the target CPU.
This patch renames softnet_data.total to softnet_data.processed: the number of
packets processed in uppper levels(IP stacks).
softnet_stat data is moved into softnet_data.
Signed-off-by: Changli Gao <xiaosuo@gmail.com>
----
include/linux/netdevice.h | 17 +++++++----------
net/core/dev.c | 26 ++++++++++++--------------
net/sched/sch_generic.c | 2 +-
3 files changed, 20 insertions(+), 25 deletions(-)
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched')
| -rw-r--r-- | net/sched/sch_generic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sched/sch_generic.c b/net/sched/sch_generic.c index aeddabf..a969b11 100644 --- a/net/sched/sch_generic.c +++ b/net/sched/sch_generic.c @@ -94,7 +94,7 @@ static inline int handle_dev_cpu_collision(struct sk_buff *skb, * Another cpu is holding lock, requeue & delay xmits for * some time. */ - __get_cpu_var(netdev_rx_stat).cpu_collision++; + __get_cpu_var(softnet_data).cpu_collision++; ret = dev_requeue_skb(skb, q); } |
