* [dpdk-stable] [PATCH v4 12/15] net/bnxt: increase the size of Rx CQ
[not found] ` <20201026035616.19264-1-ajit.khaparde@broadcom.com>
@ 2020-10-26 3:56 ` Ajit Khaparde
2020-10-26 3:56 ` [dpdk-stable] [PATCH v4 13/15] net/bnxt: fix to reset mbuf data offset Ajit Khaparde
1 sibling, 0 replies; 2+ messages in thread
From: Ajit Khaparde @ 2020-10-26 3:56 UTC (permalink / raw)
To: dev; +Cc: stable, Qingmin Liu, Randy Schacher
LRO aka TPA and jumbo frame support uses aggregation ring for placing
Rx buffers. These features can generate multiple Rx completions for a
single Rx packet. Increase size of Rx Completion Queue to handle TPA
and aggregation ring events.
Fixes: daef48efe5e5 ("net/bnxt: support set MTU")
Cc: stable@dpdk.org
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Reviewed-by: Qingmin Liu <qingmin.liu@broadcom.com>
Reviewed-by: Randy Schacher <stuart.schacher@broadcom.com>
---
drivers/net/bnxt/bnxt_ring.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/bnxt/bnxt_ring.h b/drivers/net/bnxt/bnxt_ring.h
index daf9804956..3d81f610c1 100644
--- a/drivers/net/bnxt/bnxt_ring.h
+++ b/drivers/net/bnxt/bnxt_ring.h
@@ -27,7 +27,7 @@
#define DEFAULT_RX_RING_SIZE 256
#define DEFAULT_TX_RING_SIZE 256
-#define AGG_RING_SIZE_FACTOR 2
+#define AGG_RING_SIZE_FACTOR 4
#define AGG_RING_MULTIPLIER 2
/* These assume 4k pages */
--
2.21.1 (Apple Git-122.3)
^ permalink raw reply [flat|nested] 2+ messages in thread
* [dpdk-stable] [PATCH v4 13/15] net/bnxt: fix to reset mbuf data offset
[not found] ` <20201026035616.19264-1-ajit.khaparde@broadcom.com>
2020-10-26 3:56 ` [dpdk-stable] [PATCH v4 12/15] net/bnxt: increase the size of Rx CQ Ajit Khaparde
@ 2020-10-26 3:56 ` Ajit Khaparde
1 sibling, 0 replies; 2+ messages in thread
From: Ajit Khaparde @ 2020-10-26 3:56 UTC (permalink / raw)
To: dev; +Cc: stable, Lance Richardson
Reset mbuf->data_off before handing the Rx packet to the application.
We were not doing this in the TPA path. It can cause applications
using this field for post processing to work incorrectly.
Fixes: 0958d8b6435d ("net/bnxt: support LRO")
Cc: stable@dpdk.org
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Reviewed-by: Lance Richardson <lance.richardson@broadcom.com>
---
drivers/net/bnxt/bnxt_rxr.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/bnxt/bnxt_rxr.c b/drivers/net/bnxt/bnxt_rxr.c
index 039217fa60..e41833cc43 100644
--- a/drivers/net/bnxt/bnxt_rxr.c
+++ b/drivers/net/bnxt/bnxt_rxr.c
@@ -150,6 +150,7 @@ static void bnxt_tpa_start(struct bnxt_rx_queue *rxq,
tpa_info->mbuf = mbuf;
tpa_info->len = rte_le_to_cpu_32(tpa_start->len);
+ mbuf->data_off = RTE_PKTMBUF_HEADROOM;
mbuf->nb_segs = 1;
mbuf->next = NULL;
mbuf->pkt_len = rte_le_to_cpu_32(tpa_start->len);
--
2.21.1 (Apple Git-122.3)
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-10-26 3:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CACZ4nhuC75W8Jd-FkR7qXD=E8ngOedpoE2hjYbB+fwHK6i4vSg@mail.gmail.com>
[not found] ` <20201026035616.19264-1-ajit.khaparde@broadcom.com>
2020-10-26 3:56 ` [dpdk-stable] [PATCH v4 12/15] net/bnxt: increase the size of Rx CQ Ajit Khaparde
2020-10-26 3:56 ` [dpdk-stable] [PATCH v4 13/15] net/bnxt: fix to reset mbuf data offset Ajit Khaparde
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).