diff options
author | Jesse Brandeburg <jesse.brandeburg@intel.com> | 2010-09-29 17:38:49 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-09-30 20:59:31 -0400 |
commit | 3a3b75860527a11ba5035c6aa576079245d09e2a (patch) | |
tree | 0dbd487fe8910a02c39ab4f84da6d16ebe6d29e3 /drivers/net/e1000e/82571.c | |
parent | 6af3b9ebfea38d0a057fbd7bdcecc61ac5662f02 (diff) |
e1000e: use hardware writeback batching
Most e1000e parts support batching writebacks. The problem with this is
that when some of the TADV or TIDV timers are not set, Tx can sit forever.
This is solved in this patch with write flushes using the Flush Partial
Descriptors (FPD) bit in TIDV and RDTR.
This improves bus utilization and removes partial writes on e1000e,
particularly from 82571 parts in S5500 chipset based machines.
Only ES2LAN and 82571/2 parts are included in this optimization, to reduce
testing load.
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Tested-by: Emil Tantilov <emil.s.tantilov@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/e1000e/82571.c')
-rw-r--r-- | drivers/net/e1000e/82571.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/net/e1000e/82571.c b/drivers/net/e1000e/82571.c index d3d4a57e2450..ca663f19d7df 100644 --- a/drivers/net/e1000e/82571.c +++ b/drivers/net/e1000e/82571.c | |||
@@ -1801,7 +1801,8 @@ struct e1000_info e1000_82571_info = { | |||
1801 | | FLAG_RESET_OVERWRITES_LAA /* errata */ | 1801 | | FLAG_RESET_OVERWRITES_LAA /* errata */ |
1802 | | FLAG_TARC_SPEED_MODE_BIT /* errata */ | 1802 | | FLAG_TARC_SPEED_MODE_BIT /* errata */ |
1803 | | FLAG_APME_CHECK_PORT_B, | 1803 | | FLAG_APME_CHECK_PORT_B, |
1804 | .flags2 = FLAG2_DISABLE_ASPM_L1, /* errata 13 */ | 1804 | .flags2 = FLAG2_DISABLE_ASPM_L1 /* errata 13 */ |
1805 | | FLAG2_DMA_BURST, | ||
1805 | .pba = 38, | 1806 | .pba = 38, |
1806 | .max_hw_frame_size = DEFAULT_JUMBO, | 1807 | .max_hw_frame_size = DEFAULT_JUMBO, |
1807 | .get_variants = e1000_get_variants_82571, | 1808 | .get_variants = e1000_get_variants_82571, |
@@ -1819,7 +1820,8 @@ struct e1000_info e1000_82572_info = { | |||
1819 | | FLAG_RX_CSUM_ENABLED | 1820 | | FLAG_RX_CSUM_ENABLED |
1820 | | FLAG_HAS_CTRLEXT_ON_LOAD | 1821 | | FLAG_HAS_CTRLEXT_ON_LOAD |
1821 | | FLAG_TARC_SPEED_MODE_BIT, /* errata */ | 1822 | | FLAG_TARC_SPEED_MODE_BIT, /* errata */ |
1822 | .flags2 = FLAG2_DISABLE_ASPM_L1, /* errata 13 */ | 1823 | .flags2 = FLAG2_DISABLE_ASPM_L1 /* errata 13 */ |
1824 | | FLAG2_DMA_BURST, | ||
1823 | .pba = 38, | 1825 | .pba = 38, |
1824 | .max_hw_frame_size = DEFAULT_JUMBO, | 1826 | .max_hw_frame_size = DEFAULT_JUMBO, |
1825 | .get_variants = e1000_get_variants_82571, | 1827 | .get_variants = e1000_get_variants_82571, |