net/mlx5e: Add headroom only to the first fragment in legacy RQ
authorMaxim Mikityanskiy <maximmi@nvidia.com>
Wed, 26 Jan 2022 13:49:23 +0000 (15:49 +0200)
committerSaeed Mahameed <saeedm@nvidia.com>
Thu, 17 Mar 2022 18:51:54 +0000 (11:51 -0700)
Currently, rq->buff.headroom is applied to all fragments in legacy RQ.
In the linear mode, there is a non-zero headroom, but there is only one
fragment per packet. In the non-linear mode, the headroom is zero.

This commit changes the logic to apply the headroom only to the first
fragment. The current behavior remains the same for both linear and
non-linear modes. However, it allows the next commit to enable headroom
for the non-linear mode, which will be applied only to the first
fragment.

Signed-off-by: Maxim Mikityanskiy <maximmi@nvidia.com>
Reviewed-by: Tariq Toukan <tariqt@nvidia.com>
Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
drivers/net/ethernet/mellanox/mlx5/core/en_rx.c

index 074a44b..6eda906 100644 (file)
@@ -373,12 +373,15 @@ static int mlx5e_alloc_rx_wqe(struct mlx5e_rq *rq, struct mlx5e_rx_wqe_cyc *wqe,
        int i;
 
        for (i = 0; i < rq->wqe.info.num_frags; i++, frag++) {
+               u16 headroom;
+
                err = mlx5e_get_rx_frag(rq, frag);
                if (unlikely(err))
                        goto free_frags;
 
+               headroom = i == 0 ? rq->buff.headroom : 0;
                wqe->data[i].addr = cpu_to_be64(frag->di->addr +
-                                               frag->offset + rq->buff.headroom);
+                                               frag->offset + headroom);
        }
 
        return 0;