tcp: remove unneeded code from tcp_stream_alloc_skb()
authorEric Dumazet <edumazet@google.com>
Mon, 25 Oct 2021 22:13:42 +0000 (15:13 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 26 Oct 2021 13:45:12 +0000 (14:45 +0100)
Aligning @size argument to 4 bytes is not needed.

The header alignment has nothing to do with @size.

It really depends on skb->head alignment and MAX_TCP_HEADER.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Acked-by: Soheil Hassas Yeganeh <soheil@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp.c

index f30a268..d0b848f 100644 (file)
@@ -861,9 +861,6 @@ struct sk_buff *tcp_stream_alloc_skb(struct sock *sk, int size, gfp_t gfp,
 {
        struct sk_buff *skb;
 
-       /* The TCP header must be at least 32-bit aligned.  */
-       size = ALIGN(size, 4);
-
        if (unlikely(tcp_under_memory_pressure(sk)))
                sk_mem_reclaim_partial(sk);