[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20150711.211714.1422141941490741042.davem@davemloft.net>
Date: Sat, 11 Jul 2015 21:17:14 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: fw@...len.de
Cc: netdev@...r.kernel.org, edumazet@...gle.com
Subject: Re: [PATCH -next] Revert "ipv4: use skb coalescing in
defragmentation"
From: Florian Westphal <fw@...len.de>
Date: Sat, 11 Jul 2015 01:37:36 +0200
> This reverts commit 3cc4949269e01f39443d0fcfffb5bc6b47878d45.
>
> There is nothing wrong with coalescing during defragmentation, it
> reduces truesize overhead and simplifies things for the receiving
> socket (no fraglist walk needed).
>
> However, it also destroys geometry of the original fragments.
> While that doesn't cause any breakage (we make sure to not exceed largest
> original size) ip_do_fragment contains a 'fastpath' that takes advantage
> of a present frag list and results in fragments that (in most cases)
> match what was received.
>
> In case its needed the coalescing could be done later, when we're sure
> the skb is not forwarded. But discussion during NFWS resulted in
> 'lets just remove this for now'.
>
> Cc: Eric Dumazet <edumazet@...gle.com>
> Signed-off-by: Florian Westphal <fw@...len.de>
Applied, thanks for following up on this.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists