i40e/i40evf: Remove VF Rx csum offload for tunneled packets
authoralice michael <alice.michael@intel.com>
Thu, 6 Apr 2017 09:59:34 +0000 (05:59 -0400)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Wed, 19 Apr 2017 23:32:53 +0000 (16:32 -0700)
Rx checksum offload for tunneled packets was never being negotiated or
requested by VF. This capability was assumed by default and enabled in
current hardware for VF. Going forward, this feature needs to be disabled
or advanced ptypes should be negotiated with PF in the future.

Change-ID: I9e54cfa8a90e03ab6956db4412f1e337ccd2c2e0
Signed-off-by: Preethi Banala <preethi.banala@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/i40evf/i40e_txrx.c

index 460171edc412e88eab3cbbbf48cf0e41c6e8dc43..fc4b14af370dd677ec5fd88ae6ccfc3ee3f27a87 100644 (file)
@@ -831,13 +831,6 @@ static inline void i40e_rx_checksum(struct i40e_vsi *vsi,
        if (rx_error & BIT(I40E_RX_DESC_ERROR_PPRS_SHIFT))
                return;
 
-       /* If there is an outer header present that might contain a checksum
-        * we need to bump the checksum level by 1 to reflect the fact that
-        * we are indicating we validated the inner checksum.
-        */
-       if (decoded.tunnel_type >= I40E_RX_PTYPE_TUNNEL_IP_GRENAT)
-               skb->csum_level = 1;
-
        /* Only report checksum unnecessary for TCP, UDP, or SCTP */
        switch (decoded.inner_prot) {
        case I40E_RX_PTYPE_INNER_PROT_TCP: