aboutsummaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorDavid S. Miller <davem@davemloft.net>2005-07-05 18:20:27 -0400
committerDavid S. Miller <davem@davemloft.net>2005-07-05 18:20:27 -0400
commitb4e26f5ea0dbdd1e813c5571fb467022d8eb948a (patch)
tree6d1d36cafebb126ff3946443e6d03fec60776f5e /net
parentaa93466bdfd901b926e033801f0b82b3eaa67be2 (diff)
[TCP]: Fix send-side cpu utiliziation regression.
Only put user data purely to pages when doing TSO. The extra page allocations cause two problems: 1) Add the overhead of the page allocations themselves. 2) Make us do small user copies when we get to the end of the TCP socket cache page. It is still beneficial to purely use pages for TSO, so we will do it for that case. Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/ipv4/tcp.c13
1 files changed, 11 insertions, 2 deletions
diff --git a/net/ipv4/tcp.c b/net/ipv4/tcp.c
index be354155b2f9..2ba73bf3a8f9 100644
--- a/net/ipv4/tcp.c
+++ b/net/ipv4/tcp.c
@@ -756,8 +756,17 @@ static inline int select_size(struct sock *sk, struct tcp_sock *tp)
756{ 756{
757 int tmp = tp->mss_cache_std; 757 int tmp = tp->mss_cache_std;
758 758
759 if (sk->sk_route_caps & NETIF_F_SG) 759 if (sk->sk_route_caps & NETIF_F_SG) {
760 tmp = 0; 760 if (sk->sk_route_caps & NETIF_F_TSO)
761 tmp = 0;
762 else {
763 int pgbreak = SKB_MAX_HEAD(MAX_TCP_HEADER);
764
765 if (tmp >= pgbreak &&
766 tmp <= pgbreak + (MAX_SKB_FRAGS - 1) * PAGE_SIZE)
767 tmp = pgbreak;
768 }
769 }
761 770
762 return tmp; 771 return tmp;
763} 772}