Thread (9 messages) 9 messages, 4 authors, 2021-07-30

Re: [dpdk-dev] [PATCH] net/ena: enable multi segment in Tx offload flags

From: Michał Krawczyk <hidden>
Date: 2021-07-14 10:02:46

pon., 12 lip 2021 o 19:03 Ghalem Boudour [off-list ref] napisał(a):
The DPDK ENA driver does not provide multi-segment tx offload capability.
Let's add DEV_TX_OFFLOAD_MULTI_SEGS to ports offload capability by default.
Hi Ghalem,

This patch enables announcement of the DEV_TX_OFFLOAD_MULTI_SEGS
capability, but still the application may not request this offload.

As ENA PMD currently assumes all the mbufs may have multiple segments
(and we don't have fast-path for the other cases), I suggest
overwriting this flag in the ena_dev_configure(), similar to what
we're doing with the DEV_RX_OFFLOAD_RSS_HASH flag.

Best regards,
Michal
quoted hunk ↗ jump to hunk
This flag in not listed in doc/guides/nics/features/default.ini, so
ena.ini does not need to be updated.

Signed-off-by: Ghalem Boudour <redacted>
Reviewed-by: Olivier Matz <redacted>
---
 drivers/net/ena/ena_ethdev.c | 1 +
 1 file changed, 1 insertion(+)
diff --git a/drivers/net/ena/ena_ethdev.c b/drivers/net/ena/ena_ethdev.c
index dfe68279fa7b..a8422ee425ce 100644
--- a/drivers/net/ena/ena_ethdev.c
+++ b/drivers/net/ena/ena_ethdev.c
@@ -2055,6 +2055,7 @@ static int ena_infos_get(struct rte_eth_dev *dev,
                        DEV_RX_OFFLOAD_TCP_CKSUM;

        rx_feat |= DEV_RX_OFFLOAD_JUMBO_FRAME;
+       tx_feat |= DEV_TX_OFFLOAD_MULTI_SEGS;

        /* Inform framework about available features */
        dev_info->rx_offload_capa = rx_feat;
--
2.28.0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help