Thread (161 messages) 161 messages, 13 authors, 2013-11-21

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

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help