[PATCH net-next 10/17] tcp: drop unnecessary local var in collapse

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ilpo Järvinen
Date: Saturday, February 28, 2009 - 7:44 am

From: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>

Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
---
 net/ipv4/tcp_output.c |    7 ++-----
 1 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c
index 77af7fa..61445b5 100644
--- a/net/ipv4/tcp_output.c
+++ b/net/ipv4/tcp_output.c
@@ -1767,11 +1767,9 @@ static void tcp_collapse_retrans(struct sock *sk, struct sk_buff *skb)
 	struct tcp_sock *tp = tcp_sk(sk);
 	struct sk_buff *next_skb = tcp_write_queue_next(sk, skb);
 	int skb_size, next_skb_size;
-	u16 flags;
 
 	skb_size = skb->len;
 	next_skb_size = next_skb->len;
-	flags = TCP_SKB_CB(skb)->flags;
 
 	BUG_ON(tcp_skb_pcount(skb) != 1 || tcp_skb_pcount(next_skb) != 1);
 
@@ -1791,9 +1789,8 @@ static void tcp_collapse_retrans(struct sock *sk, struct sk_buff *skb)
 	/* Update sequence range on original skb. */
 	TCP_SKB_CB(skb)->end_seq = TCP_SKB_CB(next_skb)->end_seq;
 
-	/* Merge over control information. */
-	flags |= TCP_SKB_CB(next_skb)->flags; /* This moves PSH/FIN etc. over */
-	TCP_SKB_CB(skb)->flags = flags;
+	/* Merge over control information. This moves PSH/FIN etc. over */
+	TCP_SKB_CB(skb)->flags |= TCP_SKB_CB(next_skb)->flags;
 
 	/* All done, get rid of second SKB and account for it so
 	 * packet counting does not break.
-- 
1.5.6.5

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/17]: tcp &amp; more tcp, Ilpo Järvinen, (Sat Feb 28, 7:44 am)
[PATCH net-2.6 01/17] tcp: fix retrans_out leaks, Ilpo Järvinen, (Sat Feb 28, 7:44 am)
[PATCH net-next 02/17] tcp: don't backtrack to sacked skbs, Ilpo Järvinen, (Sat Feb 28, 7:44 am)
[PATCH net-next 03/17] tcp: fix lost_cnt_hint miscounts, Ilpo Järvinen, (Sat Feb 28, 7:44 am)
[PATCH net-next 10/17] tcp: drop unnecessary local var in ..., Ilpo Järvinen, (Sat Feb 28, 7:44 am)
[PATCH net-next 11/17] htcp: merge icsk_ca_state compare, Ilpo Järvinen, (Sat Feb 28, 7:44 am)
[PATCH net-next 12/17] tcp: add helper for AI algorithm, Ilpo Järvinen, (Sat Feb 28, 7:44 am)
[PATCH net-next 14/17] tcp: remove pointless .dsack code, Ilpo Järvinen, (Sat Feb 28, 7:44 am)
Re: [PATCH net-2.6 01/17] tcp: fix retrans_out leaks, David Miller, (Sun Mar 1, 1:22 am)