mlxsw: spectrum: Don't count internal TX header bytes to stats
authorNogah Frankel <nogahf@mellanox.com>
Fri, 17 Jun 2016 13:09:05 +0000 (15:09 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sat, 18 Jun 2016 04:57:53 +0000 (21:57 -0700)
Stop the SW TX counter from counting the TX header bytes
since they are not being sent out.

Fixes: 56ade8fe3fe1 ("mlxsw: spectrum: Add initial support for Spectrum ASIC")
Reviewed-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: Nogah Frankel <nogahf@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlxsw/spectrum.c

index 6f9e3ddff4a8df98dd95818db7e722aaf8cb76a9..660429ebfbe16b4ca9ff6fc6ae92fc280bd2f1be 100644 (file)
@@ -408,7 +408,11 @@ static netdev_tx_t mlxsw_sp_port_xmit(struct sk_buff *skb,
        }
 
        mlxsw_sp_txhdr_construct(skb, &tx_info);
-       len = skb->len;
+       /* TX header is consumed by HW on the way so we shouldn't count its
+        * bytes as being sent.
+        */
+       len = skb->len - MLXSW_TXHDR_LEN;
+
        /* Due to a race we might fail here because of a full queue. In that
         * unlikely case we simply drop the packet.
         */