From: Ajit Khaparde Date: Mon, 8 Jan 2018 20:24:26 +0000 (-0800) Subject: net/bnxt: fix double increment of idx during Tx ring alloc X-Git-Tag: spdx-start~377 X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=1005d96445e9df72d9a907e0c82843528efed939;p=dpdk.git net/bnxt: fix double increment of idx during Tx ring alloc We are incrementing idx twice while allocating Tx rings. Since this is passed to the firmware, it may cause unexpected behavior. Fixes: daef48efe5e5 ("net/bnxt: support set MTU") Cc: stable@dpdk.org Signed-off-by: Ajit Khaparde --- diff --git a/drivers/net/bnxt/bnxt_ring.c b/drivers/net/bnxt/bnxt_ring.c index 0fa2f0c0d5..e0ca0e7f37 100644 --- a/drivers/net/bnxt/bnxt_ring.c +++ b/drivers/net/bnxt/bnxt_ring.c @@ -362,9 +362,6 @@ int bnxt_alloc_hwrm_rings(struct bnxt *bp) struct bnxt_ring *ring = txr->tx_ring_struct; unsigned int idx = i + 1 + bp->rx_cp_nr_rings; - /* Account for AGG Rings. AGG ring cnt = Rx Cmpl ring cnt */ - idx += bp->rx_cp_nr_rings; - /* Tx cmpl */ rc = bnxt_hwrm_ring_alloc(bp, cp_ring, HWRM_RING_ALLOC_INPUT_RING_TYPE_L2_CMPL,