datapath: Clean up netdev_get_stats() very slightly.
authorBen Pfaff <blp@nicira.com>
Thu, 4 Nov 2010 21:00:54 +0000 (14:00 -0700)
committerBen Pfaff <blp@nicira.com>
Tue, 9 Nov 2010 21:48:35 +0000 (13:48 -0800)
dev_get_stats() is documented to return its output argument, so there's no
need to maintain a separate pointer variable.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>
datapath/vport-netdev.c

index fce9b3978df8cd638cc900b464300319190cc875..bc1637b2564595d65c5736954ffcf003d7e88c49 100644 (file)
@@ -211,22 +211,22 @@ struct kobject *netdev_get_kobj(const struct vport *vport)
 int netdev_get_stats(const struct vport *vport, struct odp_vport_stats *stats)
 {
        const struct netdev_vport *netdev_vport = netdev_vport_priv(vport);
-       struct rtnl_link_stats64 *netdev_stats, storage;
-
-       netdev_stats = dev_get_stats(netdev_vport->dev, &storage);
-
-       stats->rx_bytes         = netdev_stats->rx_bytes;
-       stats->rx_packets       = netdev_stats->rx_packets;
-       stats->tx_bytes         = netdev_stats->tx_bytes;
-       stats->tx_packets       = netdev_stats->tx_packets;
-       stats->rx_dropped       = netdev_stats->rx_dropped;
-       stats->rx_errors        = netdev_stats->rx_errors;
-       stats->rx_frame_err     = netdev_stats->rx_frame_errors;
-       stats->rx_over_err      = netdev_stats->rx_over_errors;
-       stats->rx_crc_err       = netdev_stats->rx_crc_errors;
-       stats->tx_dropped       = netdev_stats->tx_dropped;
-       stats->tx_errors        = netdev_stats->tx_errors;
-       stats->collisions       = netdev_stats->collisions;
+       struct rtnl_link_stats64 netdev_stats;
+
+       dev_get_stats(netdev_vport->dev, &netdev_stats);
+
+       stats->rx_bytes         = netdev_stats.rx_bytes;
+       stats->rx_packets       = netdev_stats.rx_packets;
+       stats->tx_bytes         = netdev_stats.tx_bytes;
+       stats->tx_packets       = netdev_stats.tx_packets;
+       stats->rx_dropped       = netdev_stats.rx_dropped;
+       stats->rx_errors        = netdev_stats.rx_errors;
+       stats->rx_frame_err     = netdev_stats.rx_frame_errors;
+       stats->rx_over_err      = netdev_stats.rx_over_errors;
+       stats->rx_crc_err       = netdev_stats.rx_crc_errors;
+       stats->tx_dropped       = netdev_stats.tx_dropped;
+       stats->tx_errors        = netdev_stats.tx_errors;
+       stats->collisions       = netdev_stats.collisions;
 
        return 0;
 }