net/bnxt: set HW coalescing parameters
Checks
Commit Message
Set coalescing parameters correctly for Rx completion rings.
This is not being done for the Rx completion rings currently.
Fixes: 657c2a7f1dd4 ("net/bnxt: create aggregation rings when needed")
Cc: stable@dpdk.org
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Signed-off-by: Somnath Kotur <somnath.kotur@broadcom.com>
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
---
drivers/net/bnxt/bnxt_ring.c | 2 ++
1 file changed, 2 insertions(+)
Comments
On Tue, Feb 8, 2022 at 9:57 PM Ajit Khaparde <ajit.khaparde@broadcom.com> wrote:
>
> Set coalescing parameters correctly for Rx completion rings.
> This is not being done for the Rx completion rings currently.
>
> Fixes: 657c2a7f1dd4 ("net/bnxt: create aggregation rings when needed")
> Cc: stable@dpdk.org
>
> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> Signed-off-by: Somnath Kotur <somnath.kotur@broadcom.com>
> Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Patch applied to dpdk-next-net-brcm.
> ---
> drivers/net/bnxt/bnxt_ring.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/net/bnxt/bnxt_ring.c b/drivers/net/bnxt/bnxt_ring.c
> index 5c6c27fed7..4cdbb177d9 100644
> --- a/drivers/net/bnxt/bnxt_ring.c
> +++ b/drivers/net/bnxt/bnxt_ring.c
> @@ -752,6 +752,8 @@ int bnxt_alloc_hwrm_rings(struct bnxt *bp)
> rc = bnxt_alloc_hwrm_rx_ring(bp, i);
> if (rc)
> goto err_out;
> + bnxt_hwrm_set_ring_coal(bp, &coal,
> + rxq->cp_ring->cp_ring_struct->fw_ring_id);
> }
>
> /* If something is wrong with Rx ring alloc, skip Tx ring alloc */
> --
> 2.32.0 (Apple Git-132)
>
@@ -752,6 +752,8 @@ int bnxt_alloc_hwrm_rings(struct bnxt *bp)
rc = bnxt_alloc_hwrm_rx_ring(bp, i);
if (rc)
goto err_out;
+ bnxt_hwrm_set_ring_coal(bp, &coal,
+ rxq->cp_ring->cp_ring_struct->fw_ring_id);
}
/* If something is wrong with Rx ring alloc, skip Tx ring alloc */