veth: Fix the byte counters
authorEric W. Biederman <ebiederm@aristanetworks.com>
Tue, 22 Mar 2011 01:24:53 +0000 (18:24 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 22 Mar 2011 01:24:53 +0000 (18:24 -0700)
commit675071a2ef3f4a6d25ee002a7437d50431168344
tree54399646ea81261585fa8889b6d744cec0a8a3a8
parent9d2a8fa96a44ba242de3a6f56acaef7a40a97b97
veth: Fix the byte counters

Commit 44540960 "veth: move loopback logic to common location" introduced
a bug in the packet counters.  I don't understand why that happened as it
is not explained in the comments and the mut check in dev_forward_skb
retains the assumption that skb->len is the total length of the packet.

I just measured this emperically by setting up a veth pair between two
noop network namespaces setting and attempting a telnet connection between
the two.  I saw three packets in each direction and the byte counters were
exactly 14*3 = 42 bytes high in each direction.  I got the actual
packet lengths with tcpdump.

So remove the extra ETH_HLEN from the veth byte count totals.

Signed-off-by: Eric W. Biederman <ebiederm@aristanetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/veth.c