Skip to content

Commit

Permalink
net: rx_dropped accounting
Browse files Browse the repository at this point in the history
Under load, netif_rx() can drop incoming packets but administrators dont
have a chance to spot which device needs some tuning (RPS activation for
example)

This patch adds rx_dropped accounting in vlans and tunnels.

Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Eric Dumazet authored and David S. Miller committed Sep 20, 2010
1 parent 462fb2a commit 8990f46
Show file tree
Hide file tree
Showing 7 changed files with 28 additions and 8 deletions.
2 changes: 2 additions & 0 deletions net/8021q/vlan.h
Original file line number Diff line number Diff line change
Expand Up @@ -25,13 +25,15 @@ struct vlan_priority_tci_mapping {
* @rx_multicast: number of received multicast packets
* @syncp: synchronization point for 64bit counters
* @rx_errors: number of errors
* @rx_dropped: number of dropped packets
*/
struct vlan_rx_stats {
u64 rx_packets;
u64 rx_bytes;
u64 rx_multicast;
struct u64_stats_sync syncp;
unsigned long rx_errors;
unsigned long rx_dropped;
};

/**
Expand Down
9 changes: 7 additions & 2 deletions net/8021q/vlan_dev.c
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,10 @@ int vlan_skb_recv(struct sk_buff *skb, struct net_device *dev,
}
}

netif_rx(skb);
if (unlikely(netif_rx(skb) == NET_RX_DROP)) {
if (rx_stats)
rx_stats->rx_dropped++;
}
rcu_read_unlock();
return NET_RX_SUCCESS;

Expand Down Expand Up @@ -843,13 +846,15 @@ static struct rtnl_link_stats64 *vlan_dev_get_stats64(struct net_device *dev, st
accum.rx_packets += rxpackets;
accum.rx_bytes += rxbytes;
accum.rx_multicast += rxmulticast;
/* rx_errors is an ulong, not protected by syncp */
/* rx_errors, rx_dropped are ulong, not protected by syncp */
accum.rx_errors += p->rx_errors;
accum.rx_dropped += p->rx_dropped;
}
stats->rx_packets = accum.rx_packets;
stats->rx_bytes = accum.rx_bytes;
stats->rx_errors = accum.rx_errors;
stats->multicast = accum.rx_multicast;
stats->rx_dropped = accum.rx_dropped;
}
return stats;
}
Expand Down
6 changes: 4 additions & 2 deletions net/ipv4/ip_gre.c
Original file line number Diff line number Diff line change
Expand Up @@ -647,9 +647,11 @@ static int ipgre_rcv(struct sk_buff *skb)
skb_reset_network_header(skb);
ipgre_ecn_decapsulate(iph, skb);

netif_rx(skb);
if (netif_rx(skb) == NET_RX_DROP)
stats->rx_dropped++;

rcu_read_unlock();
return(0);
return 0;
}
icmp_send(skb, ICMP_DEST_UNREACH, ICMP_PORT_UNREACH, 0);

Expand Down
5 changes: 4 additions & 1 deletion net/ipv4/ipip.c
Original file line number Diff line number Diff line change
Expand Up @@ -377,7 +377,10 @@ static int ipip_rcv(struct sk_buff *skb)
skb_tunnel_rx(skb, tunnel->dev);

ipip_ecn_decapsulate(iph, skb);
netif_rx(skb);

if (netif_rx(skb) == NET_RX_DROP)
tunnel->dev->stats.rx_dropped++;

rcu_read_unlock();
return 0;
}
Expand Down
5 changes: 4 additions & 1 deletion net/ipv6/ip6_tunnel.c
Original file line number Diff line number Diff line change
Expand Up @@ -725,7 +725,10 @@ static int ip6_tnl_rcv(struct sk_buff *skb, __u16 protocol,
skb_tunnel_rx(skb, t->dev);

dscp_ecn_decapsulate(t, ipv6h, skb);
netif_rx(skb);

if (netif_rx(skb) == NET_RX_DROP)
t->dev->stats.rx_dropped++;

rcu_read_unlock();
return 0;
}
Expand Down
4 changes: 3 additions & 1 deletion net/ipv6/ip6mr.c
Original file line number Diff line number Diff line change
Expand Up @@ -666,7 +666,9 @@ static int pim6_rcv(struct sk_buff *skb)

skb_tunnel_rx(skb, reg_dev);

netif_rx(skb);
if (netif_rx(skb) == NET_RX_DROP)
reg_dev->stats.rx_dropped++;

dev_put(reg_dev);
return 0;
drop:
Expand Down
5 changes: 4 additions & 1 deletion net/ipv6/sit.c
Original file line number Diff line number Diff line change
Expand Up @@ -564,7 +564,10 @@ static int ipip6_rcv(struct sk_buff *skb)
skb_tunnel_rx(skb, tunnel->dev);

ipip6_ecn_decapsulate(iph, skb);
netif_rx(skb);

if (netif_rx(skb) == NET_RX_DROP)
tunnel->dev->stats.rx_dropped++;

rcu_read_unlock();
return 0;
}
Expand Down

0 comments on commit 8990f46

Please sign in to comment.