i40e/i40evf: Clean up feature flags

The feature flags list for i40e and i40evf is beginning to become pretty
massive.  I plan to add another 4 or so features to these drivers and
duplicating the flags for each and every flags list is becoming a bit
repetitive.

The primary change here is that we now build our features list around
hw_encap_features.  After that we assign that to vlan_features,
hw_features, and finally map that onto features.  In addition we end up
throwing features onto hw_encap_features that end up having no effect such
as the Rx offloads and SCTP_CRC.  However that should have no impact and
makes things a bit easier for us as hw_encap_features is one of the less
updated features maps available.

For i40evf I went through and sanity checked a few features as well.
Specifically RXCSUM was being set as a read-only feature which didn't make
much sense.  I have updated things so we can clear the NETIF_F_RXCSUM flag
since that is really a software feature and not a hardware one anyway so
disabling it is just a matter of ignoring the result from the hardware.

Signed-off-by: Alexander Duyck <aduyck@mirantis.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>

1 files changed, 32 insertions, 34 deletions

diff --git a/drivers/net/ethernet/intel/i40evf/i40evf_main.c b/drivers/net/ethernet/intel/i40evf/i40evf_main.c
index 9110319a8f00..e3857d890cfb 100644
--- a/drivers/net/ethernet/intel/i40evf/i40evf_main.c
+++ b/drivers/net/ethernet/intel/i40evf/i40evf_main.c
@@ -2337,40 +2337,38 @@ int i40evf_process_config(struct i40evf_adapter *adapter)
                 return -ENODEV;
         }
 
-        netdev->features |= NETIF_F_HIGHDMA |
-                            NETIF_F_SG |
-                            NETIF_F_IP_CSUM |
-                            NETIF_F_SCTP_CRC |
-                            NETIF_F_IPV6_CSUM |
-                            NETIF_F_TSO |
-                            NETIF_F_TSO6 |
-                            NETIF_F_TSO_ECN |
-                            NETIF_F_GSO_GRE |
-                            NETIF_F_GSO_UDP_TUNNEL |
-                            NETIF_F_RXCSUM |
-                            NETIF_F_GRO;
-
-        netdev->hw_enc_features |= NETIF_F_IP_CSUM             |
-                                   NETIF_F_IPV6_CSUM           |
-                                   NETIF_F_TSO                 |
-                                   NETIF_F_TSO6                |
-                                   NETIF_F_TSO_ECN             |
-                                   NETIF_F_GSO_GRE             |
-                                   NETIF_F_GSO_UDP_TUNNEL      |
-                                   NETIF_F_GSO_UDP_TUNNEL_CSUM;
-
-        if (adapter->flags & I40EVF_FLAG_OUTER_UDP_CSUM_CAPABLE)
-                netdev->features |= NETIF_F_GSO_UDP_TUNNEL_CSUM;
-
-        /* always clear VLAN features because they can change at every reset */
-        netdev->features &= ~(I40EVF_VLAN_FEATURES);
-        /* copy netdev features into list of user selectable features */
-        netdev->hw_features |= netdev->features;
-
-        if (vfres->vf_offload_flags & I40E_VIRTCHNL_VF_OFFLOAD_VLAN) {
-                netdev->vlan_features = netdev->features;
-                netdev->features |= I40EVF_VLAN_FEATURES;
-        }
+        netdev->hw_enc_features |= NETIF_F_SG                   |
+                                   NETIF_F_IP_CSUM              |
+                                   NETIF_F_IPV6_CSUM            |
+                                   NETIF_F_HIGHDMA              |
+                                   NETIF_F_SOFT_FEATURES        |
+                                   NETIF_F_TSO                  |
+                                   NETIF_F_TSO_ECN              |
+                                   NETIF_F_TSO6                 |
+                                   NETIF_F_GSO_GRE              |
+                                   NETIF_F_GSO_UDP_TUNNEL       |
+                                   NETIF_F_GSO_UDP_TUNNEL_CSUM  |
+                                   NETIF_F_SCTP_CRC             |
+                                   NETIF_F_RXHASH               |
+                                   NETIF_F_RXCSUM               |
+                                   0;
+
+        if (!(adapter->flags & I40EVF_FLAG_OUTER_UDP_CSUM_CAPABLE))
+                netdev->hw_enc_features ^= NETIF_F_GSO_UDP_TUNNEL_CSUM;
+
+        /* record features VLANs can make use of */
+        netdev->vlan_features |= netdev->hw_enc_features;
+
+        /* Write features and hw_features separately to avoid polluting
+         * with, or dropping, features that are set when we registgered.
+         */
+        netdev->hw_features |= netdev->hw_enc_features;
+
+        netdev->features |= netdev->hw_enc_features | I40EVF_VLAN_FEATURES;
+
+        /* disable VLAN features if not supported */
+        if (!(vfres->vf_offload_flags & I40E_VIRTCHNL_VF_OFFLOAD_VLAN))
+                netdev->features ^= I40EVF_VLAN_FEATURES;
 
         adapter->vsi.id = adapter->vsi_res->vsi_id;