Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4a5f4dd8 authored by Yevgeny Petrilin's avatar Yevgeny Petrilin Committed by David S. Miller
Browse files

mlx4_en: Remove FCS bytes from packet length. 



When HW doesn't remove FCS bytes they are reported in the completion
byte count, we don't need to take them to skb.

Signed-off-by: default avatarYevgeny Petrilin <yevgenyp@mellanox.co.il>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 06236ac3

drivers/net/ethernet/mellanox/mlx4/en_rx.c

+5 −1
Original line number Diff line number Diff line
@@ -581,6 +581,7 @@ int mlx4_en_process_rx_cq(struct net_device *dev, struct mlx4_en_cq *cq, int bud 
		 * Packet is OK - process it.

		 */

		length = be32_to_cpu(cqe->byte_cnt);

		length -= ring->fcs_del;

		ring->bytes += length;

		ring->packets++;
@@ -813,8 +814,11 @@ static int mlx4_en_config_rss_qp(struct mlx4_en_priv *priv, int qpn, 
	context->db_rec_addr = cpu_to_be64(ring->wqres.db.dma);



	/* Cancel FCS removal if FW allows */

	if (mdev->dev->caps.flags & MLX4_DEV_CAP_FLAG_FCS_KEEP)

	if (mdev->dev->caps.flags & MLX4_DEV_CAP_FLAG_FCS_KEEP) {

		context->param3 |= cpu_to_be32(1 << 29);

		ring->fcs_del = ETH_FCS_LEN;

	} else

		ring->fcs_del = 0;



	err = mlx4_qp_to_ready(mdev->dev, &ring->wqres.mtt, context, qp, state);

	if (err) {

drivers/net/ethernet/mellanox/mlx4/mlx4_en.h

+1 −0
Original line number Diff line number Diff line
@@ -272,6 +272,7 @@ struct mlx4_en_rx_ring { 
	u32 prod;

	u32 cons;

	u32 buf_size;

	u8  fcs_del;

	void *buf;

	void *rx_info;

	unsigned long bytes;