Skip to content

Commit

Permalink
e1000: Simple optimizations in e1000_xmit_frame
Browse files Browse the repository at this point in the history
Some simple optimizations in e1000_xmit_frame.

Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
  • Loading branch information
Krishna Kumar authored and David S. Miller committed Oct 10, 2007
1 parent e2f036d commit 6d1e3aa
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions drivers/net/e1000/e1000_main.c
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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:
Expand Down

0 comments on commit 6d1e3aa

Please sign in to comment.