d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

net/bnxt: fix representor data path

Browse Source 
1.Representor Rx ring producer index was not getting reset in
the ring full case. Fix it by incrementing only in
success case.
2.Instead of calling the mbuf specific routine to free the mbuf when
representor ring is full rte_free was being called leading to
'invalid memory' errors being logged.
3. Do not account the pkt meant for the representor in the parent
Rx ring's array that is returned to the application.

Fixes: 6dc83230b4 ("net/bnxt: support port representor data path")
Cc: stable@dpdk.org

Signed-off-by: Somnath Kotur <somnath.kotur@broadcom.com>
Reviewed-by: Sriharsha Basavapatna <sriharsha.basavapatna@broadcom.com>
Reviewed-by: Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>
Reviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
This commit is contained in:
Somnath Kotur 2020-09-10 18:56:00 -07:00 committed by Ferruh Yigit
parent 9b842a93e2
commit 526e0b0200
2 changed files with 17 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/net/bnxt/bnxt_reps.c
View File

@ -55,15 +55,17 @@ bnxt_vfr_recv(uint16_t port_id, uint16_t queue_id, struct rte_mbuf *mbuf)
mask = rep_rxr->rx_ring_struct->ring_mask;
/* Put this mbuf on the RxQ of the Representor */
prod_rx_buf = &rep_rxr->rx_buf_ring[rep_rxr->rx_prod++ & mask];
prod_rx_buf = &rep_rxr->rx_buf_ring[rep_rxr->rx_prod & mask];
if (!*prod_rx_buf) {
*prod_rx_buf = mbuf;
vfr_bp->rx_bytes[que] += mbuf->pkt_len;
vfr_bp->rx_pkts[que]++;
rep_rxr->rx_prod++;
} else {
/* Representor Rx ring full, drop pkt */
vfr_bp->rx_drop_bytes[que] += mbuf->pkt_len;
vfr_bp->rx_drop_pkts[que]++;
rte_pktmbuf_free(mbuf); /* Representor Rx ring full, drop pkt */
rte_pktmbuf_free(mbuf);
}
return 0;

27
drivers/net/bnxt/bnxt_rxr.c
View File

@ -795,6 +795,19 @@ static int bnxt_rx_pkt(struct rte_mbuf **rx_pkt,
goto rx;
}
rxr->rx_prod = prod;
if (BNXT_TRUFLOW_EN(bp) && (BNXT_VF_IS_TRUSTED(bp) || BNXT_PF(bp)) &&
vfr_flag) {
bnxt_vfr_recv(mark_id, rxq->queue_id, mbuf);
/* Now return an error so that nb_rx_pkts is not
* incremented.
* This packet was meant to be given to the representor.
* So no need to account the packet and give it to
* parent Rx burst function.
*/
rc = -ENODEV;
goto next_rx;
}
/*
* All MBUFs are allocated with the same size under DPDK,
* no optimization for rx_copy_thresh
@ -802,20 +815,6 @@ static int bnxt_rx_pkt(struct rte_mbuf **rx_pkt,
rx:
*rx_pkt = mbuf;
if (BNXT_TRUFLOW_EN(bp) &&
(BNXT_VF_IS_TRUSTED(bp) || BNXT_PF(bp)) &&
vfr_flag) {
if (!bnxt_vfr_recv(mark_id, rxq->queue_id, mbuf)) {
/* Now return an error so that nb_rx_pkts is not
* incremented.
* This packet was meant to be given to the representor.
* So no need to account the packet and give it to
* parent Rx burst function.
*/
rc = -ENODEV;
}
}
next_rx:
*raw_cons = tmp_raw_cons;