Commit 64050cda authored by Aya Levin's avatar Aya Levin Committed by Saeed Mahameed
Browse files

Revert "net/mlx5e: Block offload of outer header csum for UDP tunnels" 



This reverts commit 6d6727dd.

Although the NIC doesn't support offload of outer header CSUM, using
gso_partial_features allows offloading the tunnel's segmentation. The
driver relies on the stack CSUM calculation of the outer header. For
this, NETIF_F_GSO_UDP_TUNNEL_CSUM must be a member of the device's
features.

Fixes: 6d6727dd ("net/mlx5e: Block offload of outer header csum for UDP tunnels")
Signed-off-by: default avatarAya Levin <ayal@nvidia.com>
Reviewed-by: default avatarGal Pressman <gal@nvidia.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@nvidia.com>
parent 9e72a55a

drivers/net/ethernet/mellanox/mlx5/core/en_main.c

+7 −3
Original line number Diff line number Diff line
@@ -4789,9 +4789,13 @@ static void mlx5e_build_nic_netdev(struct net_device *netdev) 
	}



	if (mlx5_vxlan_allowed(mdev->vxlan) || mlx5_geneve_tx_allowed(mdev)) {

		netdev->hw_features     |= NETIF_F_GSO_UDP_TUNNEL;

		netdev->hw_enc_features |= NETIF_F_GSO_UDP_TUNNEL;

		netdev->vlan_features |= NETIF_F_GSO_UDP_TUNNEL;

		netdev->hw_features     |= NETIF_F_GSO_UDP_TUNNEL |

					   NETIF_F_GSO_UDP_TUNNEL_CSUM;

		netdev->hw_enc_features |= NETIF_F_GSO_UDP_TUNNEL |

					   NETIF_F_GSO_UDP_TUNNEL_CSUM;

		netdev->gso_partial_features = NETIF_F_GSO_UDP_TUNNEL_CSUM;

		netdev->vlan_features |= NETIF_F_GSO_UDP_TUNNEL |

					 NETIF_F_GSO_UDP_TUNNEL_CSUM;

	}



	if (mlx5e_tunnel_proto_supported_tx(mdev, IPPROTO_GRE)) {