Commit cf3677ae authored by Pavel Emelyanov's avatar Pavel Emelyanov Committed by David S. Miller

ipmr: Use on-device stats instead of private ones.

These devices use the private area of appropriate size for
statistics. Turning them to use on-device ones make them
"privless" and thus - really small wrt kmalloc cache, they
are allocated from.

Besides, code looks nicer, because of absence of multi-braced
type casts and dereferences.

[ Fix build failures -DaveM ]
Signed-off-by: default avatarPavel Emelyanov <xemul@openvz.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3dca02af
...@@ -181,26 +181,20 @@ static int reg_vif_num = -1; ...@@ -181,26 +181,20 @@ static int reg_vif_num = -1;
static int reg_vif_xmit(struct sk_buff *skb, struct net_device *dev) static int reg_vif_xmit(struct sk_buff *skb, struct net_device *dev)
{ {
read_lock(&mrt_lock); read_lock(&mrt_lock);
((struct net_device_stats*)netdev_priv(dev))->tx_bytes += skb->len; dev->stats.tx_bytes += skb->len;
((struct net_device_stats*)netdev_priv(dev))->tx_packets++; dev->stats.tx_packets++;
ipmr_cache_report(skb, reg_vif_num, IGMPMSG_WHOLEPKT); ipmr_cache_report(skb, reg_vif_num, IGMPMSG_WHOLEPKT);
read_unlock(&mrt_lock); read_unlock(&mrt_lock);
kfree_skb(skb); kfree_skb(skb);
return 0; return 0;
} }
static struct net_device_stats *reg_vif_get_stats(struct net_device *dev)
{
return (struct net_device_stats*)netdev_priv(dev);
}
static void reg_vif_setup(struct net_device *dev) static void reg_vif_setup(struct net_device *dev)
{ {
dev->type = ARPHRD_PIMREG; dev->type = ARPHRD_PIMREG;
dev->mtu = ETH_DATA_LEN - sizeof(struct iphdr) - 8; dev->mtu = ETH_DATA_LEN - sizeof(struct iphdr) - 8;
dev->flags = IFF_NOARP; dev->flags = IFF_NOARP;
dev->hard_start_xmit = reg_vif_xmit; dev->hard_start_xmit = reg_vif_xmit;
dev->get_stats = reg_vif_get_stats;
dev->destructor = free_netdev; dev->destructor = free_netdev;
} }
...@@ -209,8 +203,7 @@ static struct net_device *ipmr_reg_vif(void) ...@@ -209,8 +203,7 @@ static struct net_device *ipmr_reg_vif(void)
struct net_device *dev; struct net_device *dev;
struct in_device *in_dev; struct in_device *in_dev;
dev = alloc_netdev(sizeof(struct net_device_stats), "pimreg", dev = alloc_netdev(0, "pimreg", reg_vif_setup);
reg_vif_setup);
if (dev == NULL) if (dev == NULL)
return NULL; return NULL;
...@@ -1170,8 +1163,8 @@ static void ipmr_queue_xmit(struct sk_buff *skb, struct mfc_cache *c, int vifi) ...@@ -1170,8 +1163,8 @@ static void ipmr_queue_xmit(struct sk_buff *skb, struct mfc_cache *c, int vifi)
if (vif->flags & VIFF_REGISTER) { if (vif->flags & VIFF_REGISTER) {
vif->pkt_out++; vif->pkt_out++;
vif->bytes_out+=skb->len; vif->bytes_out+=skb->len;
((struct net_device_stats*)netdev_priv(vif->dev))->tx_bytes += skb->len; vif->dev->stats.tx_bytes += skb->len;
((struct net_device_stats*)netdev_priv(vif->dev))->tx_packets++; vif->dev->stats.tx_packets++;
ipmr_cache_report(skb, vifi, IGMPMSG_WHOLEPKT); ipmr_cache_report(skb, vifi, IGMPMSG_WHOLEPKT);
kfree_skb(skb); kfree_skb(skb);
return; return;
...@@ -1487,8 +1480,8 @@ int pim_rcv_v1(struct sk_buff * skb) ...@@ -1487,8 +1480,8 @@ int pim_rcv_v1(struct sk_buff * skb)
skb->pkt_type = PACKET_HOST; skb->pkt_type = PACKET_HOST;
dst_release(skb->dst); dst_release(skb->dst);
skb->dst = NULL; skb->dst = NULL;
((struct net_device_stats*)netdev_priv(reg_dev))->rx_bytes += skb->len; reg_dev->stats.rx_bytes += skb->len;
((struct net_device_stats*)netdev_priv(reg_dev))->rx_packets++; reg_dev->stats.rx_packets++;
nf_reset(skb); nf_reset(skb);
netif_rx(skb); netif_rx(skb);
dev_put(reg_dev); dev_put(reg_dev);
...@@ -1542,8 +1535,8 @@ static int pim_rcv(struct sk_buff * skb) ...@@ -1542,8 +1535,8 @@ static int pim_rcv(struct sk_buff * skb)
skb->ip_summed = 0; skb->ip_summed = 0;
skb->pkt_type = PACKET_HOST; skb->pkt_type = PACKET_HOST;
dst_release(skb->dst); dst_release(skb->dst);
((struct net_device_stats*)netdev_priv(reg_dev))->rx_bytes += skb->len; reg_dev->stats.rx_bytes += skb->len;
((struct net_device_stats*)netdev_priv(reg_dev))->rx_packets++; reg_dev->stats.rx_packets++;
skb->dst = NULL; skb->dst = NULL;
nf_reset(skb); nf_reset(skb);
netif_rx(skb); netif_rx(skb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment