From 04c881eb6441fff2e91c9b9e23502bc554c0f437 Mon Sep 17 00:00:00 2001 From: Andy Zhou Date: Mon, 28 Apr 2014 15:46:30 -0700 Subject: [PATCH] netdev-linux: favor netlink stats for physical ports Currently physical ports stats are collected from kernel datapath. However, those counter do not reflect actual wire packet counters when GSO, TSO or GRO are enabled by the NIC. In the meantime, the stats collected form routing stack does. While both stats are valid, Reporting kernel netdev stats for packet counts and byte counts make it easier to correlate those numbers with external measurements. Signed-off-by: Andy Zhou Acked-by: Ben Pfaff --- lib/netdev-linux.c | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c index 5c7065fe1..c1d93237e 100644 --- a/lib/netdev-linux.c +++ b/lib/netdev-linux.c @@ -1563,9 +1563,17 @@ netdev_linux_get_stats(const struct netdev *netdev_, error = 0; } } else if (netdev->vport_stats_error) { - /* stats not available from OVS then use ioctl stats. */ + /* stats not available from OVS then use netdev stats. */ *stats = dev_stats; } else { + /* Use kernel netdev's packet and byte counts since vport's counters + * do not reflect packet counts on the wire when GSO, TSO or GRO are + * enabled. */ + stats->rx_packets = dev_stats.rx_packets; + stats->rx_bytes = dev_stats.rx_bytes; + stats->tx_packets = dev_stats.tx_packets; + stats->tx_bytes = dev_stats.tx_bytes; + stats->rx_errors += dev_stats.rx_errors; stats->tx_errors += dev_stats.tx_errors; stats->rx_dropped += dev_stats.rx_dropped; @@ -1629,6 +1637,14 @@ netdev_tap_get_stats(const struct netdev *netdev_, struct netdev_stats *stats) stats->tx_heartbeat_errors = 0; stats->tx_window_errors = 0; } else { + /* Use kernel netdev's packet and byte counts since vport counters + * do not reflect packet counts on the wire when GSO, TSO or GRO + * are enabled. */ + stats->rx_packets = dev_stats.tx_packets; + stats->rx_bytes = dev_stats.tx_bytes; + stats->tx_packets = dev_stats.rx_packets; + stats->tx_bytes = dev_stats.rx_bytes; + stats->rx_dropped += dev_stats.tx_dropped; stats->tx_dropped += dev_stats.rx_dropped; -- 2.43.0