tcp: reindent two spots after prequeue removal
authorFlorian Westphal <fw@strlen.de>
Sun, 30 Jul 2017 01:57:19 +0000 (03:57 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 31 Jul 2017 21:37:49 +0000 (14:37 -0700)
These two branches are now always true, remove the conditional.
objdiff shows no changes.

Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_input.c

index 770ce6cb3eca728b1d3ce8b543c56387fcc5bf64..87efde9f5a907cfb536fd83089c15b34a0b9b826 100644 (file)
@@ -4611,16 +4611,14 @@ static void tcp_data_queue(struct sock *sk, struct sk_buff *skb)
                        goto out_of_window;
 
                /* Ok. In sequence. In window. */
-               if (eaten <= 0) {
 queue_and_out:
-                       if (eaten < 0) {
-                               if (skb_queue_len(&sk->sk_receive_queue) == 0)
-                                       sk_forced_mem_schedule(sk, skb->truesize);
-                               else if (tcp_try_rmem_schedule(sk, skb, skb->truesize))
-                                       goto drop;
-                       }
-                       eaten = tcp_queue_rcv(sk, skb, 0, &fragstolen);
+               if (eaten < 0) {
+                       if (skb_queue_len(&sk->sk_receive_queue) == 0)
+                               sk_forced_mem_schedule(sk, skb->truesize);
+                       else if (tcp_try_rmem_schedule(sk, skb, skb->truesize))
+                               goto drop;
                }
+               eaten = tcp_queue_rcv(sk, skb, 0, &fragstolen);
                tcp_rcv_nxt_update(tp, TCP_SKB_CB(skb)->end_seq);
                if (skb->len)
                        tcp_event_data_recv(sk, skb);
@@ -5410,30 +5408,28 @@ void tcp_rcv_established(struct sock *sk, struct sk_buff *skb,
                        int eaten = 0;
                        bool fragstolen = false;
 
-                       if (!eaten) {
-                               if (tcp_checksum_complete(skb))
-                                       goto csum_error;
+                       if (tcp_checksum_complete(skb))
+                               goto csum_error;
 
-                               if ((int)skb->truesize > sk->sk_forward_alloc)
-                                       goto step5;
+                       if ((int)skb->truesize > sk->sk_forward_alloc)
+                               goto step5;
 
-                               /* Predicted packet is in window by definition.
-                                * seq == rcv_nxt and rcv_wup <= rcv_nxt.
-                                * Hence, check seq<=rcv_wup reduces to:
-                                */
-                               if (tcp_header_len ==
-                                   (sizeof(struct tcphdr) + TCPOLEN_TSTAMP_ALIGNED) &&
-                                   tp->rcv_nxt == tp->rcv_wup)
-                                       tcp_store_ts_recent(tp);
+                       /* Predicted packet is in window by definition.
+                        * seq == rcv_nxt and rcv_wup <= rcv_nxt.
+                        * Hence, check seq<=rcv_wup reduces to:
+                        */
+                       if (tcp_header_len ==
+                           (sizeof(struct tcphdr) + TCPOLEN_TSTAMP_ALIGNED) &&
+                           tp->rcv_nxt == tp->rcv_wup)
+                               tcp_store_ts_recent(tp);
 
-                               tcp_rcv_rtt_measure_ts(sk, skb);
+                       tcp_rcv_rtt_measure_ts(sk, skb);
 
-                               NET_INC_STATS(sock_net(sk), LINUX_MIB_TCPHPHITS);
+                       NET_INC_STATS(sock_net(sk), LINUX_MIB_TCPHPHITS);
 
-                               /* Bulk data transfer: receiver */
-                               eaten = tcp_queue_rcv(sk, skb, tcp_header_len,
-                                                     &fragstolen);
-                       }
+                       /* Bulk data transfer: receiver */
+                       eaten = tcp_queue_rcv(sk, skb, tcp_header_len,
+                                             &fragstolen);
 
                        tcp_event_data_recv(sk, skb);