Bug 593801 - [RHEL5.5] TCP bandwidth problems with TPA and bnx2x cards
Summary: [RHEL5.5] TCP bandwidth problems with TPA and bnx2x cards
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: kernel
Version: 5.5
Hardware: All
OS: Linux
medium
medium
Target Milestone: rc
: ---
Assignee: Stanislaw Gruszka
QA Contact: Network QE
URL:
Whiteboard:
Depends On:
Blocks: 613900
TreeView+ depends on / blocked
 
Reported: 2010-05-19 18:04 UTC by Flavio Leitner
Modified: 2018-10-27 12:04 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-01-13 21:32:59 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2011:0017 0 normal SHIPPED_LIVE Important: Red Hat Enterprise Linux 5.6 kernel security and bug fix update 2011-01-13 10:37:42 UTC

Comment 2 Flavio Leitner 2010-05-19 18:13:36 UTC
Public summary:
---------------

The TCP bandwidth is very low when using bnx2x with TPA. After analysis
done at the customer site, the following patch has been identified as
a fix.

Author: Herbert Xu <herbert.org.au>  2006-09-01 01:11:02
Committer: David S. Miller <davem.net>  2006-09-23 01:18:44
Parent: 3015d5d4e5b15eddea272a697e83391100581932 ([RTNETLINK]: Fix typo causing
wrong skb to be freed)
Child:  a9917c06652165fe4eeb9ab7a5d1e0674e90e508 ([XFRM] STATE: Fix flusing
with hash mask.)
Branches: coal, fixes, link, master, remotes/origin/master, tx
Follows: v2.6.18
Precedes: v2.6.19-rc1

  [TCP]: Fix rcv mss estimate for LRO

  By passing a Linux-generated TSO packet straight back into Linux, Xen
  becomes our first LRO user :) Unfortunately, there is at least one spot
  in our stack that needs to be changed to cope with this.

  The receive MSS estimate is computed from the raw packet size.  This is
  broken if the packet is GSO/LRO.  Fortunately the real MSS can be found
  in gso_size so we simply need to use that if it is non-zero.

  Real LRO NICs should of course set the gso_size field in future.

  Signed-off-by: Herbert Xu <herbert.org.au>
  Signed-off-by: David S. Miller <davem>

diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c
index caf3c41..511b738 100644
--- a/net/ipv4/tcp_input.c
+++ b/net/ipv4/tcp_input.c
@@ -127,7 +127,7 @@ static void tcp_measure_rcv_mss(struct sock *sk,
      /* skb->len may jitter because of SACKs, even if peer
       * sends good full-sized frames.
       */
-       len = skb->len;
+       len = skb_shinfo(skb)->gso_size ?: skb->len;
      if (len >= icsk->icsk_ack.rcv_mss) {
              icsk->icsk_ack.rcv_mss = len;
      } else {

Comment 3 RHEL Program Management 2010-06-03 11:28:00 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release.  Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products.  This request is not yet committed for inclusion in an Update
release.

Comment 8 Jarod Wilson 2010-07-12 15:46:44 UTC
in kernel-2.6.18-206.el5
You can download this test kernel from http://people.redhat.com/jwilson/el5

Detailed testing feedback is always welcomed.

Comment 18 errata-xmlrpc 2011-01-13 21:32:59 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2011-0017.html


Note You need to log in before you can comment on or make changes to this bug.