diff options
author | Jesse Brandeburg <jesse.brandeburg@intel.com> | 2009-02-24 19:37:31 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-02-24 19:37:31 -0500 |
commit | 32344a394029baeca5bcc9fa839694b23a82cc64 (patch) | |
tree | 47360c3dd12fcc7aa100547ee3becbe9d0961a3d /drivers/net/ixgbe/ixgbe_main.c | |
parent | c7e4358ae8e49f85a4d528366046da26b36a712f (diff) |
ixbge: fix bug when using large pages and jumbo frames
it was pointed out on the list that ixgbe was failing when using 64kB pages
and large 16kB MTU.
since with a 64kB PAGE_SIZE MAX_SKB_FRAGS = 3, the way the driver was
configuring page usage was assuming 2kB is half a page, and was only
ever dmaing that much data to a half page.
(16kB - header size) / 2048 = 7 or 8 pages, which would far exceed 3
adjust the driver to account for these large pages, the hardware can
support DMA to up to 16kB for each descriptor.
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@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/ixgbe/ixgbe_main.c')
-rw-r--r-- | drivers/net/ixgbe/ixgbe_main.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/drivers/net/ixgbe/ixgbe_main.c b/drivers/net/ixgbe/ixgbe_main.c index e0d736cc245b..65642355a836 100644 --- a/drivers/net/ixgbe/ixgbe_main.c +++ b/drivers/net/ixgbe/ixgbe_main.c | |||
@@ -1550,7 +1550,14 @@ static void ixgbe_configure_srrctl(struct ixgbe_adapter *adapter, int index) | |||
1550 | srrctl &= ~IXGBE_SRRCTL_BSIZEPKT_MASK; | 1550 | srrctl &= ~IXGBE_SRRCTL_BSIZEPKT_MASK; |
1551 | 1551 | ||
1552 | if (adapter->flags & IXGBE_FLAG_RX_PS_ENABLED) { | 1552 | if (adapter->flags & IXGBE_FLAG_RX_PS_ENABLED) { |
1553 | srrctl |= IXGBE_RXBUFFER_2048 >> IXGBE_SRRCTL_BSIZEPKT_SHIFT; | 1553 | u16 bufsz = IXGBE_RXBUFFER_2048; |
1554 | /* grow the amount we can receive on large page machines */ | ||
1555 | if (bufsz < (PAGE_SIZE / 2)) | ||
1556 | bufsz = (PAGE_SIZE / 2); | ||
1557 | /* cap the bufsz at our largest descriptor size */ | ||
1558 | bufsz = min((u16)IXGBE_MAX_RXBUFFER, bufsz); | ||
1559 | |||
1560 | srrctl |= bufsz >> IXGBE_SRRCTL_BSIZEPKT_SHIFT; | ||
1554 | srrctl |= IXGBE_SRRCTL_DESCTYPE_HDR_SPLIT_ALWAYS; | 1561 | srrctl |= IXGBE_SRRCTL_DESCTYPE_HDR_SPLIT_ALWAYS; |
1555 | srrctl |= ((IXGBE_RX_HDR_SIZE << | 1562 | srrctl |= ((IXGBE_RX_HDR_SIZE << |
1556 | IXGBE_SRRCTL_BSIZEHDRSIZE_SHIFT) & | 1563 | IXGBE_SRRCTL_BSIZEHDRSIZE_SHIFT) & |