diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2008-09-01 12:46:10 +0100 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-09-03 09:53:43 -0400 |
commit | c264361d31bec0a444e0c4ffd40479e66eef6c15 (patch) | |
tree | 5b454610641bf3e4404f2a919b2e6ef516db63f4 /drivers/net | |
parent | 5b39fe307b58fc6d71265eeed809ab799a2e2ec9 (diff) | |
download | op-kernel-dev-c264361d31bec0a444e0c4ffd40479e66eef6c15.zip op-kernel-dev-c264361d31bec0a444e0c4ffd40479e66eef6c15.tar.gz |
sfc: XMAC statistics fix-ups
Exclude assumed size of RX control frames from rx_bad_bytes.
Exclude assumed size of TX control frames from tx_good_bytes for
consistency with rx_good_bytes.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/sfc/falcon_xmac.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/net/sfc/falcon_xmac.c b/drivers/net/sfc/falcon_xmac.c index 55c0d97..ab114b4 100644 --- a/drivers/net/sfc/falcon_xmac.c +++ b/drivers/net/sfc/falcon_xmac.c @@ -554,9 +554,11 @@ void falcon_update_stats_xmac(struct efx_nic *efx) /* Update derived statistics */ mac_stats->tx_good_bytes = - (mac_stats->tx_bytes - mac_stats->tx_bad_bytes); + (mac_stats->tx_bytes - mac_stats->tx_bad_bytes - + mac_stats->tx_control * 64); mac_stats->rx_bad_bytes = - (mac_stats->rx_bytes - mac_stats->rx_good_bytes); + (mac_stats->rx_bytes - mac_stats->rx_good_bytes - + mac_stats->rx_control * 64); } int falcon_check_xmac(struct efx_nic *efx) |