Commit 92df7b51 authored by David S. Miller's avatar David S. Miller

[TCP]: tcp_write_xmit() tabbing cleanup

Put the main basic block of work at the top-level of
tabbing, and mark the TCP_CLOSE test with unlikely().
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a762a980
...@@ -842,15 +842,17 @@ static inline void tcp_cwnd_validate(struct sock *sk, struct tcp_sock *tp) ...@@ -842,15 +842,17 @@ static inline void tcp_cwnd_validate(struct sock *sk, struct tcp_sock *tp)
static int tcp_write_xmit(struct sock *sk, int nonagle) static int tcp_write_xmit(struct sock *sk, int nonagle)
{ {
struct tcp_sock *tp = tcp_sk(sk); struct tcp_sock *tp = tcp_sk(sk);
struct sk_buff *skb;
unsigned int mss_now; unsigned int mss_now;
int sent_pkts;
/* If we are closed, the bytes will have to remain here. /* If we are closed, the bytes will have to remain here.
* In time closedown will finish, we empty the write queue and all * In time closedown will finish, we empty the write queue and all
* will be happy. * will be happy.
*/ */
if (sk->sk_state != TCP_CLOSE) { if (unlikely(sk->sk_state == TCP_CLOSE))
struct sk_buff *skb; return 0;
int sent_pkts = 0;
/* Account for SACKS, we may need to fragment due to this. /* Account for SACKS, we may need to fragment due to this.
* It is just like the real MSS changing on us midstream. * It is just like the real MSS changing on us midstream.
...@@ -858,7 +860,7 @@ static int tcp_write_xmit(struct sock *sk, int nonagle) ...@@ -858,7 +860,7 @@ static int tcp_write_xmit(struct sock *sk, int nonagle)
* IP options mid-stream. Silly to do, but cover it. * IP options mid-stream. Silly to do, but cover it.
*/ */
mss_now = tcp_current_mss(sk, 1); mss_now = tcp_current_mss(sk, 1);
sent_pkts = 0;
while ((skb = sk->sk_send_head) && while ((skb = sk->sk_send_head) &&
tcp_snd_test(sk, skb, mss_now, tcp_snd_test(sk, skb, mss_now,
tcp_skb_is_last(sk, skb) ? nonagle : tcp_skb_is_last(sk, skb) ? nonagle :
...@@ -888,8 +890,6 @@ static int tcp_write_xmit(struct sock *sk, int nonagle) ...@@ -888,8 +890,6 @@ static int tcp_write_xmit(struct sock *sk, int nonagle)
} }
return !tp->packets_out && sk->sk_send_head; return !tp->packets_out && sk->sk_send_head;
}
return 0;
} }
/* Push out any pending frames which were held back due to /* Push out any pending frames which were held back due to
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment