diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2009-06-05 00:04:16 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-06-08 03:21:48 -0400 |
commit | 042a53a9e437feaf2230dd2cadcecfae9c7bfe05 (patch) | |
tree | ae9078f61e390a3014aecb3fe80d3438ab25ee51 /drivers/net/e1000 | |
parent | eae3f29cc73f83cc3f1891d3ad40021b5172c630 (diff) |
net: skb_shared_info optimization
skb_dma_unmap() is quite expensive for small packets,
because we use two different cache lines from skb_shared_info.
One to access nr_frags, one to access dma_maps[0]
Instead of dma_maps being an array of MAX_SKB_FRAGS + 1 elements,
let dma_head alone in a new dma_head field, close to nr_frags,
to reduce cache lines misses.
Tested on my dev machine (bnx2 & tg3 adapters), nice speedup !
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/e1000')
-rw-r--r-- | drivers/net/e1000/e1000_main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/e1000/e1000_main.c b/drivers/net/e1000/e1000_main.c index 05e87a59f1c6..8d36743c8140 100644 --- a/drivers/net/e1000/e1000_main.c +++ b/drivers/net/e1000/e1000_main.c | |||
@@ -2998,7 +2998,7 @@ static int e1000_tx_map(struct e1000_adapter *adapter, | |||
2998 | size -= 4; | 2998 | size -= 4; |
2999 | 2999 | ||
3000 | buffer_info->length = size; | 3000 | buffer_info->length = size; |
3001 | buffer_info->dma = map[0] + offset; | 3001 | buffer_info->dma = skb_shinfo(skb)->dma_head + offset; |
3002 | buffer_info->time_stamp = jiffies; | 3002 | buffer_info->time_stamp = jiffies; |
3003 | buffer_info->next_to_watch = i; | 3003 | buffer_info->next_to_watch = i; |
3004 | 3004 | ||
@@ -3039,7 +3039,7 @@ static int e1000_tx_map(struct e1000_adapter *adapter, | |||
3039 | size -= 4; | 3039 | size -= 4; |
3040 | 3040 | ||
3041 | buffer_info->length = size; | 3041 | buffer_info->length = size; |
3042 | buffer_info->dma = map[f + 1] + offset; | 3042 | buffer_info->dma = map[f] + offset; |
3043 | buffer_info->time_stamp = jiffies; | 3043 | buffer_info->time_stamp = jiffies; |
3044 | buffer_info->next_to_watch = i; | 3044 | buffer_info->next_to_watch = i; |
3045 | 3045 | ||