Re: [PATCH v2 net-next] net: introduce gro_frag_list_enable sysctl
From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2013-10-30 02:05:49
From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2013-10-30 02:05:49
On Tue, Oct 29, 2013 at 10:02:53PM -0400, David Miller wrote:
From: Eric Dumazet <redacted> Date: Tue, 29 Oct 2013 17:53:48 -0700quoted
So should we apply the first fix to avoid the BUG_ON() ?Please be more specific, are you talking about splitting up this patch in some way?
I think Eric is referring to the patch that removes the BUG_ON in skb_segment and deals with the new mega-GRO packets. I think that's fine for stable, but for the long term we should fix it properly as these new meag-GRO packets still retain the existing packet boundaries and are trivially segmentable. If we are indeed able to do that, I doubt we would even need the sysctl patch since the GRO performance should be vastly superior to the non-GRO case, even for a router/bridge. Cheers, -- Email: Herbert Xu [off-list ref] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt