igb/ixgbe: remove unecessary checks for CHECKSUM_UNNECESSARY

Both of these drivers do a check to verify ip_summed is set to
CHECKSUM_UNNECESSARY prior to passing the packet to GRO.  GRO itself
already does such a check so it is redundant and can be removed as this
will likely cause out of order issues when receiving a packet that didn't
pass checksum validation.

Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 6 insertions, 14 deletions

diff --git a/drivers/net/igb/igb_main.c b/drivers/net/igb/igb_main.c
index 4ecf4dfce9bd..f7f861215242 100644
--- a/drivers/net/igb/igb_main.c
+++ b/drivers/net/igb/igb_main.c
@@ -4430,20 +4430,12 @@ static void igb_receive_skb(struct igb_ring *ring, u8 status,
         bool vlan_extracted = (adapter->vlgrp && (status & E1000_RXD_STAT_VP));
 
         skb_record_rx_queue(skb, ring->queue_index);
-        if (skb->ip_summed == CHECKSUM_UNNECESSARY) {
-                if (vlan_extracted)
-                        vlan_gro_receive(&ring->napi, adapter->vlgrp,
-                                         le16_to_cpu(rx_desc->wb.upper.vlan),
-                                         skb);
-                else
-                        napi_gro_receive(&ring->napi, skb);
-        } else {
-                if (vlan_extracted)
-                        vlan_hwaccel_receive_skb(skb, adapter->vlgrp,
-                                          le16_to_cpu(rx_desc->wb.upper.vlan));
-                else
-                        netif_receive_skb(skb);
-        }
+        if (vlan_extracted)
+                vlan_gro_receive(&ring->napi, adapter->vlgrp,
+                                 le16_to_cpu(rx_desc->wb.upper.vlan),
+                                 skb);
+        else
+                napi_gro_receive(&ring->napi, skb);
 }
 
 static inline void igb_rx_checksum_adv(struct igb_adapter *adapter,