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/ixgb | |
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/ixgb')
-rw-r--r-- | drivers/net/ixgb/ixgb_main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ixgb/ixgb_main.c b/drivers/net/ixgb/ixgb_main.c index 6eb7f37a113b..9c897cf86b9f 100644 --- a/drivers/net/ixgb/ixgb_main.c +++ b/drivers/net/ixgb/ixgb_main.c | |||
@@ -1300,7 +1300,7 @@ ixgb_tx_map(struct ixgb_adapter *adapter, struct sk_buff *skb, | |||
1300 | buffer_info->length = size; | 1300 | buffer_info->length = size; |
1301 | WARN_ON(buffer_info->dma != 0); | 1301 | WARN_ON(buffer_info->dma != 0); |
1302 | buffer_info->time_stamp = jiffies; | 1302 | buffer_info->time_stamp = jiffies; |
1303 | buffer_info->dma = map[0] + offset; | 1303 | buffer_info->dma = skb_shinfo(skb)->dma_head + offset; |
1304 | pci_map_single(adapter->pdev, | 1304 | pci_map_single(adapter->pdev, |
1305 | skb->data + offset, | 1305 | skb->data + offset, |
1306 | size, | 1306 | size, |
@@ -1340,7 +1340,7 @@ ixgb_tx_map(struct ixgb_adapter *adapter, struct sk_buff *skb, | |||
1340 | 1340 | ||
1341 | buffer_info->length = size; | 1341 | buffer_info->length = size; |
1342 | buffer_info->time_stamp = jiffies; | 1342 | buffer_info->time_stamp = jiffies; |
1343 | buffer_info->dma = map[f + 1] + offset; | 1343 | buffer_info->dma = map[f] + offset; |
1344 | buffer_info->next_to_watch = 0; | 1344 | buffer_info->next_to_watch = 0; |
1345 | 1345 | ||
1346 | len -= size; | 1346 | len -= size; |