Re: [PATCH v2 net-next] net: introduce gro_frag_list_enable sysctl
From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2013-10-30 04:42:18
From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2013-10-30 04:42:18
On Tue, Oct 29, 2013 at 09:34:41PM -0700, Eric Dumazet wrote:
What matters ? GRO ?
What matters is that you should not treat the forwarding case separately from the host case. For virtualisation the host case looks exactly like the forwarding case. IOW, if having a 64KB packet matters for the host, then it matters for forwarding as well.
Before my patch, GRO packets were 17 MSS, and nobody complained packets were too small, so what are you saying exactly ?
I'm not criticsing your mega-GRO patch at all. That one is great and means that we'll get aggregated packets up to 64K. What we need to do is just to patch up the GSO code so that it can handle these mega-packets properly. 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