From: =?UTF-8?q?=C3=8D=C3=B1igo=20Huguet?= <ihuguet@redhat.com>

Date: Thu, 23 Mar 2023 09:34:17 +0100

Subject: [PATCH] sfc: ef10: don't overwrite offload features at NIC reset

MIME-Version: 1.0

Content-Type: text/plain; charset=UTF-8

Content-Transfer-Encoding: 8bit

References: bsc#1012628

Patch-mainline: 6.2.10

Git-commit: ca4a80e4bb7e87daf33b27d2ab9e4f5311018a89

[ Upstream commit ca4a80e4bb7e87daf33b27d2ab9e4f5311018a89 ]

At NIC reset, some offload features related to encapsulated traffic

might have changed (this mainly happens if the firmware-variant is

changed with the sfboot userspace tool). Because of this, features are

checked and set again at reset time.

However, this was not done right, and some features were improperly

overwritten at NIC reset:

- Tunneled IPv6 segmentation was always disabled

- Features disabled with ethtool were reenabled

- Features that becomes unsupported after the reset were not disabled

Also, checking if the device supports IPV6_CSUM to enable TSO6 is no

longer necessary because all currently supported devices support it.

Additionally, move the assignment of some other features to the

EF10_OFFLOAD_FEATURES macro, like it is done in ef100, leaving the

selection of features in efx_pci_probe_post_io a bit cleaner.

Fixes: ffffd2454a7a ("sfc: correctly advertise tunneled IPv6 segmentation")

Fixes: 24b2c3751aa3 ("sfc: advertise encapsulated offloads on EF10")

Reported-by: Tianhao Zhao <tizhao@redhat.com>

Suggested-by: Jonathan Cooper <jonathan.s.cooper@amd.com>

Tested-by: Jonathan Cooper <jonathan.s.cooper@amd.com>

Signed-off-by: Íñigo Huguet <ihuguet@redhat.com>

Acked-by: Edward Cree <ecree.xilinx@gmail.com>

Link: https://lore.kernel.org/r/20230323083417.7345-1-ihuguet@redhat.com

Signed-off-by: Jakub Kicinski <kuba@kernel.org>

Signed-off-by: Sasha Levin <sashal@kernel.org>

Signed-off-by: Jiri Slaby <jslaby@suse.cz>

---

 drivers/net/ethernet/sfc/ef10.c | 38 ++++++++++++++++++++++-----------

 drivers/net/ethernet/sfc/efx.c  | 17 ++++++---------

 2 files changed, 33 insertions(+), 22 deletions(-)

diff --git a/drivers/net/ethernet/sfc/ef10.c b/drivers/net/ethernet/sfc/ef10.c

index 7022fb20..d30459db 100644

--- a/drivers/net/ethernet/sfc/ef10.c

+++ b/drivers/net/ethernet/sfc/ef10.c

@@ -1304,7 +1304,8 @@ static void efx_ef10_fini_nic(struct efx_nic *efx)

 static int efx_ef10_init_nic(struct efx_nic *efx)

 {

 	struct efx_ef10_nic_data *nic_data = efx->nic_data;

-	netdev_features_t hw_enc_features = 0;

+	struct net_device *net_dev = efx->net_dev;

+	netdev_features_t tun_feats, tso_feats;

 	int rc;

 	if (nic_data->must_check_datapath_caps) {

@@ -1349,20 +1350,30 @@ static int efx_ef10_init_nic(struct efx_nic *efx)

 		nic_data->must_restore_piobufs = false;

 	}

-	/* add encapsulated checksum offload features */

+	/* encap features might change during reset if fw variant changed */

 	if (efx_has_cap(efx, VXLAN_NVGRE) && !efx_ef10_is_vf(efx))

-		hw_enc_features |= NETIF_F_IP_CSUM | NETIF_F_IPV6_CSUM;

-	/* add encapsulated TSO features */

-	if (efx_has_cap(efx, TX_TSO_V2_ENCAP)) {

-		netdev_features_t encap_tso_features;

+		net_dev->hw_enc_features |= NETIF_F_IP_CSUM | NETIF_F_IPV6_CSUM;

+	else

+		net_dev->hw_enc_features &= ~(NETIF_F_IP_CSUM | NETIF_F_IPV6_CSUM);

-		encap_tso_features = NETIF_F_GSO_UDP_TUNNEL | NETIF_F_GSO_GRE |

-			NETIF_F_GSO_UDP_TUNNEL_CSUM | NETIF_F_GSO_GRE_CSUM;

+	tun_feats = NETIF_F_GSO_UDP_TUNNEL | NETIF_F_GSO_GRE |

+		    NETIF_F_GSO_UDP_TUNNEL_CSUM | NETIF_F_GSO_GRE_CSUM;

+	tso_feats = NETIF_F_TSO | NETIF_F_TSO6;

-		hw_enc_features |= encap_tso_features | NETIF_F_TSO;

-		efx->net_dev->features |= encap_tso_features;

+	if (efx_has_cap(efx, TX_TSO_V2_ENCAP)) {

+		/* If this is first nic_init, or if it is a reset and a new fw

+		 * variant has added new features, enable them by default.

+		 * If the features are not new, maintain their current value.

+		 */

+		if (!(net_dev->hw_features & tun_feats))

+			net_dev->features |= tun_feats;

+		net_dev->hw_enc_features |= tun_feats | tso_feats;

+		net_dev->hw_features |= tun_feats;

+	} else {

+		net_dev->hw_enc_features &= ~(tun_feats | tso_feats);

+		net_dev->hw_features &= ~tun_feats;

+		net_dev->features &= ~tun_feats;

 	}

-	efx->net_dev->hw_enc_features = hw_enc_features;

 	/* don't fail init if RSS setup doesn't work */

 	rc = efx->type->rx_push_rss_config(efx, false,

@@ -4021,7 +4032,10 @@ static unsigned int efx_ef10_recycle_ring_size(const struct efx_nic *efx)

 	 NETIF_F_HW_VLAN_CTAG_FILTER |	\

 	 NETIF_F_IPV6_CSUM |		\

 	 NETIF_F_RXHASH |		\

-	 NETIF_F_NTUPLE)

+	 NETIF_F_NTUPLE |		\

+	 NETIF_F_SG |			\

+	 NETIF_F_RXCSUM |		\

+	 NETIF_F_RXALL)

 const struct efx_nic_type efx_hunt_a0_vf_nic_type = {

 	.is_vf = true,

diff --git a/drivers/net/ethernet/sfc/efx.c b/drivers/net/ethernet/sfc/efx.c

index 3a86f121..6a1bff54 100644

--- a/drivers/net/ethernet/sfc/efx.c

+++ b/drivers/net/ethernet/sfc/efx.c

@@ -1001,21 +1001,18 @@ static int efx_pci_probe_post_io(struct efx_nic *efx)

 	}

 	/* Determine netdevice features */

-	net_dev->features |= (efx->type->offload_features | NETIF_F_SG |

-			      NETIF_F_TSO | NETIF_F_RXCSUM | NETIF_F_RXALL);

-	if (efx->type->offload_features & (NETIF_F_IPV6_CSUM | NETIF_F_HW_CSUM)) {

-		net_dev->features |= NETIF_F_TSO6;

-		if (efx_has_cap(efx, TX_TSO_V2_ENCAP))

-			net_dev->hw_enc_features |= NETIF_F_TSO6;

-	}

-	/* Check whether device supports TSO */

-	if (!efx->type->tso_versions || !efx->type->tso_versions(efx))

-		net_dev->features &= ~NETIF_F_ALL_TSO;

+	net_dev->features |= efx->type->offload_features;

+

+	/* Add TSO features */

+	if (efx->type->tso_versions && efx->type->tso_versions(efx))

+		net_dev->features |= NETIF_F_TSO | NETIF_F_TSO6;

+

 	/* Mask for features that also apply to VLAN devices */

 	net_dev->vlan_features |= (NETIF_F_HW_CSUM | NETIF_F_SG |

 				   NETIF_F_HIGHDMA | NETIF_F_ALL_TSO |

 				   NETIF_F_RXCSUM);

+	/* Determine user configurable features */

 	net_dev->hw_features |= net_dev->features & ~efx->fixed_features;

 	/* Disable receiving frames with bad FCS, by default. */

-- 

2.35.3