igb: use new eth_get_headlen interface
authorAlexander Duyck <alexander.h.duyck@intel.com>
Fri, 5 Sep 2014 23:20:43 +0000 (19:20 -0400)
committerDavid S. Miller <davem@davemloft.net>
Sat, 6 Sep 2014 00:47:02 +0000 (17:47 -0700)
Update igb to drop the igb_get_headlen function in favor of eth_get_headlen.

Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Acked-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/intel/igb/igb_main.c

index 89de7fee5e9413a59aec095ad8c44a032c138b18..4c023f0e54e4cb0d46adfcc43bf6a255bafc2738 100644 (file)
@@ -6768,113 +6768,6 @@ static bool igb_is_non_eop(struct igb_ring *rx_ring,
        return true;
 }
 
-/**
- *  igb_get_headlen - determine size of header for LRO/GRO
- *  @data: pointer to the start of the headers
- *  @max_len: total length of section to find headers in
- *
- *  This function is meant to determine the length of headers that will
- *  be recognized by hardware for LRO, and GRO offloads.  The main
- *  motivation of doing this is to only perform one pull for IPv4 TCP
- *  packets so that we can do basic things like calculating the gso_size
- *  based on the average data per packet.
- **/
-static unsigned int igb_get_headlen(unsigned char *data,
-                                   unsigned int max_len)
-{
-       union {
-               unsigned char *network;
-               /* l2 headers */
-               struct ethhdr *eth;
-               struct vlan_hdr *vlan;
-               /* l3 headers */
-               struct iphdr *ipv4;
-               struct ipv6hdr *ipv6;
-       } hdr;
-       __be16 protocol;
-       u8 nexthdr = 0; /* default to not TCP */
-       u8 hlen;
-
-       /* this should never happen, but better safe than sorry */
-       if (max_len < ETH_HLEN)
-               return max_len;
-
-       /* initialize network frame pointer */
-       hdr.network = data;
-
-       /* set first protocol and move network header forward */
-       protocol = hdr.eth->h_proto;
-       hdr.network += ETH_HLEN;
-
-       /* handle any vlan tag if present */
-       if (protocol == htons(ETH_P_8021Q)) {
-               if ((hdr.network - data) > (max_len - VLAN_HLEN))
-                       return max_len;
-
-               protocol = hdr.vlan->h_vlan_encapsulated_proto;
-               hdr.network += VLAN_HLEN;
-       }
-
-       /* handle L3 protocols */
-       if (protocol == htons(ETH_P_IP)) {
-               if ((hdr.network - data) > (max_len - sizeof(struct iphdr)))
-                       return max_len;
-
-               /* access ihl as a u8 to avoid unaligned access on ia64 */
-               hlen = (hdr.network[0] & 0x0F) << 2;
-
-               /* verify hlen meets minimum size requirements */
-               if (hlen < sizeof(struct iphdr))
-                       return hdr.network - data;
-
-               /* record next protocol if header is present */
-               if (!(hdr.ipv4->frag_off & htons(IP_OFFSET)))
-                       nexthdr = hdr.ipv4->protocol;
-       } else if (protocol == htons(ETH_P_IPV6)) {
-               if ((hdr.network - data) > (max_len - sizeof(struct ipv6hdr)))
-                       return max_len;
-
-               /* record next protocol */
-               nexthdr = hdr.ipv6->nexthdr;
-               hlen = sizeof(struct ipv6hdr);
-       } else {
-               return hdr.network - data;
-       }
-
-       /* relocate pointer to start of L4 header */
-       hdr.network += hlen;
-
-       /* finally sort out TCP */
-       if (nexthdr == IPPROTO_TCP) {
-               if ((hdr.network - data) > (max_len - sizeof(struct tcphdr)))
-                       return max_len;
-
-               /* access doff as a u8 to avoid unaligned access on ia64 */
-               hlen = (hdr.network[12] & 0xF0) >> 2;
-
-               /* verify hlen meets minimum size requirements */
-               if (hlen < sizeof(struct tcphdr))
-                       return hdr.network - data;
-
-               hdr.network += hlen;
-       } else if (nexthdr == IPPROTO_UDP) {
-               if ((hdr.network - data) > (max_len - sizeof(struct udphdr)))
-                       return max_len;
-
-               hdr.network += sizeof(struct udphdr);
-       }
-
-       /* If everything has gone correctly hdr.network should be the
-        * data section of the packet and will be the end of the header.
-        * If not then it probably represents the end of the last recognized
-        * header.
-        */
-       if ((hdr.network - data) < max_len)
-               return hdr.network - data;
-       else
-               return max_len;
-}
-
 /**
  *  igb_pull_tail - igb specific version of skb_pull_tail
  *  @rx_ring: rx descriptor ring packet is being transacted on
@@ -6919,7 +6812,7 @@ static void igb_pull_tail(struct igb_ring *rx_ring,
        /* we need the header to contain the greater of either ETH_HLEN or
         * 60 bytes if the skb->len is less than 60 for skb_pad.
         */
-       pull_len = igb_get_headlen(va, IGB_RX_HDR_LEN);
+       pull_len = eth_get_headlen(va, IGB_RX_HDR_LEN);
 
        /* align pull length to size of long to optimize memcpy performance */
        skb_copy_to_linear_data(skb, va, ALIGN(pull_len, sizeof(long)));