From e1ba2361fad11e8c01fb26ef222e2595b462f9cb Mon Sep 17 00:00:00 2001 From: Krishna Kumar Date: Tue, 8 Dec 2009 22:26:13 +0000 Subject: [PATCH] --- yaml --- r: 183232 b: refs/heads/master c: 12d50c46dc0f7fd2e625c4befaa5fa5740a7a594 h: refs/heads/master v: v3 --- [refs] | 2 +- trunk/include/net/tcp.h | 16 +++++++++------- trunk/net/ipv4/tcp_output.c | 5 ----- 3 files changed, 10 insertions(+), 13 deletions(-) diff --git a/[refs] b/[refs] index b99e1088d734..fd3608c3f30c 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 7ef8a2127a64d131c4dbe5b843e5708f6e75552b +refs/heads/master: 12d50c46dc0f7fd2e625c4befaa5fa5740a7a594 diff --git a/trunk/include/net/tcp.h b/trunk/include/net/tcp.h index 34f5cc24d903..185e22baecb1 100644 --- a/trunk/include/net/tcp.h +++ b/trunk/include/net/tcp.h @@ -856,13 +856,6 @@ static inline void tcp_check_probe_timer(struct sock *sk) icsk->icsk_rto, TCP_RTO_MAX); } -static inline void tcp_push_pending_frames(struct sock *sk) -{ - struct tcp_sock *tp = tcp_sk(sk); - - __tcp_push_pending_frames(sk, tcp_current_mss(sk), tp->nonagle); -} - static inline void tcp_init_wl(struct tcp_sock *tp, u32 seq) { tp->snd_wl1 = seq; @@ -1342,6 +1335,15 @@ static inline int tcp_write_queue_empty(struct sock *sk) return skb_queue_empty(&sk->sk_write_queue); } +static inline void tcp_push_pending_frames(struct sock *sk) +{ + if (tcp_send_head(sk)) { + struct tcp_sock *tp = tcp_sk(sk); + + __tcp_push_pending_frames(sk, tcp_current_mss(sk), tp->nonagle); + } +} + /* Start sequence of the highest skb with SACKed bit, valid only if * sacked > 0 or when the caller has ensured validity by itself. */ diff --git a/trunk/net/ipv4/tcp_output.c b/trunk/net/ipv4/tcp_output.c index 383ce237640f..12b2af36eab8 100644 --- a/trunk/net/ipv4/tcp_output.c +++ b/trunk/net/ipv4/tcp_output.c @@ -1794,11 +1794,6 @@ static int tcp_write_xmit(struct sock *sk, unsigned int mss_now, int nonagle, void __tcp_push_pending_frames(struct sock *sk, unsigned int cur_mss, int nonagle) { - struct sk_buff *skb = tcp_send_head(sk); - - if (!skb) - return; - /* If we are closed, the bytes will have to remain here. * In time closedown will finish, we empty the write queue and * all will be happy.