diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2007-12-11 14:30:32 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 17:56:33 -0500 |
commit | 1781f7f5804e52ee2d35328b129602146a8d8254 (patch) | |
tree | dc67102f480e59d87859c50392452be4f41487bd /net | |
parent | 27ab2568649d5ba6c5a20212079b7c4f6da4ca0d (diff) |
[UDP]: Restore missing inDatagrams increments
The previous move of the the UDP inDatagrams counter caused the
counting of encapsulated packets, SUNRPC data (as opposed to call)
packets and RXRPC packets to go missing.
This patch restores all of these.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/ipv4/udp.c | 6 | ||||
-rw-r--r-- | net/ipv6/proc.c | 1 | ||||
-rw-r--r-- | net/ipv6/udp.c | 2 | ||||
-rw-r--r-- | net/rxrpc/ar-input.c | 4 | ||||
-rw-r--r-- | net/sunrpc/xprtsock.c | 6 |
5 files changed, 17 insertions, 2 deletions
diff --git a/net/ipv4/udp.c b/net/ipv4/udp.c index f50de5d5218d..78cfcb4a1b3f 100644 --- a/net/ipv4/udp.c +++ b/net/ipv4/udp.c | |||
@@ -110,6 +110,7 @@ | |||
110 | */ | 110 | */ |
111 | 111 | ||
112 | DEFINE_SNMP_STAT(struct udp_mib, udp_statistics) __read_mostly; | 112 | DEFINE_SNMP_STAT(struct udp_mib, udp_statistics) __read_mostly; |
113 | EXPORT_SYMBOL(udp_statistics); | ||
113 | 114 | ||
114 | struct hlist_head udp_hash[UDP_HTABLE_SIZE]; | 115 | struct hlist_head udp_hash[UDP_HTABLE_SIZE]; |
115 | DEFINE_RWLOCK(udp_hash_lock); | 116 | DEFINE_RWLOCK(udp_hash_lock); |
@@ -969,8 +970,11 @@ int udp_queue_rcv_skb(struct sock * sk, struct sk_buff *skb) | |||
969 | int ret; | 970 | int ret; |
970 | 971 | ||
971 | ret = (*up->encap_rcv)(sk, skb); | 972 | ret = (*up->encap_rcv)(sk, skb); |
972 | if (ret <= 0) | 973 | if (ret <= 0) { |
974 | UDP_INC_STATS_BH(UDP_MIB_INDATAGRAMS, | ||
975 | is_udplite); | ||
973 | return -ret; | 976 | return -ret; |
977 | } | ||
974 | } | 978 | } |
975 | 979 | ||
976 | /* FALLTHROUGH -- it's a UDP Packet */ | 980 | /* FALLTHROUGH -- it's a UDP Packet */ |
diff --git a/net/ipv6/proc.c b/net/ipv6/proc.c index 44937616057e..41e9980b3e0e 100644 --- a/net/ipv6/proc.c +++ b/net/ipv6/proc.c | |||
@@ -27,6 +27,7 @@ | |||
27 | #include <net/ip.h> | 27 | #include <net/ip.h> |
28 | #include <net/sock.h> | 28 | #include <net/sock.h> |
29 | #include <net/tcp.h> | 29 | #include <net/tcp.h> |
30 | #include <net/udp.h> | ||
30 | #include <net/transp_v6.h> | 31 | #include <net/transp_v6.h> |
31 | #include <net/ipv6.h> | 32 | #include <net/ipv6.h> |
32 | 33 | ||
diff --git a/net/ipv6/udp.c b/net/ipv6/udp.c index 87bccec9882a..36bdcd2e1b52 100644 --- a/net/ipv6/udp.c +++ b/net/ipv6/udp.c | |||
@@ -34,6 +34,7 @@ | |||
34 | #include <linux/ipv6.h> | 34 | #include <linux/ipv6.h> |
35 | #include <linux/icmpv6.h> | 35 | #include <linux/icmpv6.h> |
36 | #include <linux/init.h> | 36 | #include <linux/init.h> |
37 | #include <linux/module.h> | ||
37 | #include <linux/skbuff.h> | 38 | #include <linux/skbuff.h> |
38 | #include <asm/uaccess.h> | 39 | #include <asm/uaccess.h> |
39 | 40 | ||
@@ -51,6 +52,7 @@ | |||
51 | #include "udp_impl.h" | 52 | #include "udp_impl.h" |
52 | 53 | ||
53 | DEFINE_SNMP_STAT(struct udp_mib, udp_stats_in6) __read_mostly; | 54 | DEFINE_SNMP_STAT(struct udp_mib, udp_stats_in6) __read_mostly; |
55 | EXPORT_SYMBOL(udp_stats_in6); | ||
54 | 56 | ||
55 | static inline int udp_v6_get_port(struct sock *sk, unsigned short snum) | 57 | static inline int udp_v6_get_port(struct sock *sk, unsigned short snum) |
56 | { | 58 | { |
diff --git a/net/rxrpc/ar-input.c b/net/rxrpc/ar-input.c index 91b5bbb003e2..f446d9b9925f 100644 --- a/net/rxrpc/ar-input.c +++ b/net/rxrpc/ar-input.c | |||
@@ -20,6 +20,7 @@ | |||
20 | #include <net/sock.h> | 20 | #include <net/sock.h> |
21 | #include <net/af_rxrpc.h> | 21 | #include <net/af_rxrpc.h> |
22 | #include <net/ip.h> | 22 | #include <net/ip.h> |
23 | #include <net/udp.h> | ||
23 | #include "ar-internal.h" | 24 | #include "ar-internal.h" |
24 | 25 | ||
25 | unsigned long rxrpc_ack_timeout = 1; | 26 | unsigned long rxrpc_ack_timeout = 1; |
@@ -707,10 +708,13 @@ void rxrpc_data_ready(struct sock *sk, int count) | |||
707 | if (skb_checksum_complete(skb)) { | 708 | if (skb_checksum_complete(skb)) { |
708 | rxrpc_free_skb(skb); | 709 | rxrpc_free_skb(skb); |
709 | rxrpc_put_local(local); | 710 | rxrpc_put_local(local); |
711 | UDP_INC_STATS_BH(UDP_MIB_INERRORS, 0); | ||
710 | _leave(" [CSUM failed]"); | 712 | _leave(" [CSUM failed]"); |
711 | return; | 713 | return; |
712 | } | 714 | } |
713 | 715 | ||
716 | UDP_INC_STATS_BH(UDP_MIB_INDATAGRAMS, 0); | ||
717 | |||
714 | /* the socket buffer we have is owned by UDP, with UDP's data all over | 718 | /* the socket buffer we have is owned by UDP, with UDP's data all over |
715 | * it, but we really want our own */ | 719 | * it, but we really want our own */ |
716 | skb_orphan(skb); | 720 | skb_orphan(skb); |
diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c index 2f630a512ab7..6fa52f44de0f 100644 --- a/net/sunrpc/xprtsock.c +++ b/net/sunrpc/xprtsock.c | |||
@@ -838,8 +838,12 @@ static void xs_udp_data_ready(struct sock *sk, int len) | |||
838 | copied = repsize; | 838 | copied = repsize; |
839 | 839 | ||
840 | /* Suck it into the iovec, verify checksum if not done by hw. */ | 840 | /* Suck it into the iovec, verify checksum if not done by hw. */ |
841 | if (csum_partial_copy_to_xdr(&rovr->rq_private_buf, skb)) | 841 | if (csum_partial_copy_to_xdr(&rovr->rq_private_buf, skb)) { |
842 | UDPX_INC_STATS_BH(sk, UDP_MIB_INERRORS); | ||
842 | goto out_unlock; | 843 | goto out_unlock; |
844 | } | ||
845 | |||
846 | UDPX_INC_STATS_BH(sk, UDP_MIB_INDATAGRAMS); | ||
843 | 847 | ||
844 | /* Something worked... */ | 848 | /* Something worked... */ |
845 | dst_confirm(skb->dst); | 849 | dst_confirm(skb->dst); |