e1000: Simple optimizations in e1000_xmit_frame

Some simple optimizations in e1000_xmit_frame.

Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>

1 files changed, 4 insertions, 5 deletions

diff --git a/drivers/net/e1000/e1000_main.c b/drivers/net/e1000/e1000_main.c
index 10505de00bcc..047263830e6a 100644
--- a/drivers/net/e1000/e1000_main.c
+++ b/drivers/net/e1000/e1000_main.c
@@ -3261,14 +3261,13 @@ e1000_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
         unsigned int first, max_per_txd = E1000_MAX_DATA_PER_TXD;
         unsigned int max_txd_pwr = E1000_MAX_TXD_PWR;
         unsigned int tx_flags = 0;
-        unsigned int len = skb->len;
+        unsigned int len = skb->len - skb->data_len;
         unsigned long flags;
-        unsigned int nr_frags = 0;
-        unsigned int mss = 0;
+        unsigned int nr_frags;
+        unsigned int mss;
         int count = 0;
         int tso;
         unsigned int f;
-        len -= skb->data_len;
 
         /* This goes back to the question of how to logically map a tx queue
          * to a flow.  Right now, performance is impacted slightly negatively
@@ -3302,7 +3301,7 @@ e1000_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
                 * points to just header, pull a few bytes of payload from
                 * frags into skb->data */
                 hdr_len = skb_transport_offset(skb) + tcp_hdrlen(skb);
-                if (skb->data_len && (hdr_len == (skb->len - skb->data_len))) {
+                if (skb->data_len && hdr_len == len) {
                         switch (adapter->hw.mac_type) {
                                 unsigned int pull_size;
                         case e1000_82544: