On 04/02/2020 21:28, David Marchand wrote:
> The check on TSO capability did not ensure ip checksum, tcp checksum and
> TSO tx offloads were available which resulted in a port init failure
> (example below with a ena device):
> 
> *2020-02-04T17:42:52.976Z|00084|dpdk|ERR|Ethdev port_id=0 requested Tx
> offloads 0x2a doesn't match Tx offloads capabilities 0xe in
> rte_eth_dev_configure()*
> 
> Fixes: 29cf9c1b3b9c ("userspace: Add TCP Segmentation Offload support")
> 
> Reported-by: Ravi Kerur <rke...@gmail.com>
> Signed-off-by: David Marchand <david.march...@redhat.com>
> ---
>  lib/netdev-dpdk.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
> index b108cbd6b..eb1a7af94 100644
> --- a/lib/netdev-dpdk.c
> +++ b/lib/netdev-dpdk.c
> @@ -1132,7 +1132,7 @@ dpdk_eth_dev_init(struct netdev_dpdk *dev)
>          dev->hw_ol_features &= ~NETDEV_RX_HW_SCATTER;
>      }
>  
> -    if (info.tx_offload_capa & tx_tso_offload_capa) {
> +    if ((info.tx_offload_capa & tx_tso_offload_capa) == tx_tso_offload_capa) 
> {
>          dev->hw_ol_features |= NETDEV_TX_TSO_OFFLOAD;
>      } else {
>          dev->hw_ol_features &= ~NETDEV_TX_TSO_OFFLOAD;
> 

Acked-by: Kevin Traynor <ktray...@redhat.com>

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to