The TCP_TSO tx offload was missing from the conversion function. This was 
effectively taking the request for that offload out of consideration when 
selecting the tx function. As a result a vector path which does not support TSO 
could be chosen when that offload was requested, when instead the scalar path 
should be chosen which does support that offload. Fix this by adding the TSO 
offload to the conversion function.

Fixes: c008a5e740bd ("common/idpf: add queue setup/release")
Cc: [email protected]

Signed-off-by: Ciara Loftus <[email protected]>
---
 drivers/net/intel/idpf/idpf_rxtx.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/net/intel/idpf/idpf_rxtx.c 
b/drivers/net/intel/idpf/idpf_rxtx.c
index 1c725065df..4796d8b862 100644
--- a/drivers/net/intel/idpf/idpf_rxtx.c
+++ b/drivers/net/intel/idpf/idpf_rxtx.c
@@ -42,6 +42,8 @@ idpf_tx_offload_convert(uint64_t offload)
                ol |= IDPF_TX_OFFLOAD_TCP_CKSUM;
        if ((offload & RTE_ETH_TX_OFFLOAD_SCTP_CKSUM) != 0)
                ol |= IDPF_TX_OFFLOAD_SCTP_CKSUM;
+       if ((offload & RTE_ETH_TX_OFFLOAD_TCP_TSO) != 0)
+               ol |= IDPF_TX_OFFLOAD_TCP_TSO;
        if ((offload & RTE_ETH_TX_OFFLOAD_MULTI_SEGS) != 0)
                ol |= IDPF_TX_OFFLOAD_MULTI_SEGS;
        if ((offload & RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE) != 0)
--

Looks good to me,
Acked-by: Praveen Shetty <[email protected]>

2.34.1

Reply via email to