netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hannes Frederic Sowa <hannes@stressinduktion.org>
To: Eric Dumazet <eric.dumazet@gmail.com>
Cc: Francois WELLENREITER <f.wellenreiter@gmail.com>, netdev@vger.kernel.org
Subject: Re: Does IPv6 support Jumbograms ?
Date: Thu, 10 Apr 2014 15:28:42 +0200	[thread overview]
Message-ID: <20140410132842.GI27255@order.stressinduktion.org> (raw)
In-Reply-To: <1397090170.16584.21.camel@edumazet-glaptop2.roam.corp.google.com>

On Wed, Apr 09, 2014 at 05:36:10PM -0700, Eric Dumazet wrote:
> On Thu, 2014-04-10 at 00:35 +0200, Hannes Frederic Sowa wrote:
> > On Wed, Apr 09, 2014 at 02:41:05PM -0700, Eric Dumazet wrote:
> > > What do you mean by blocked ?
> > > 
> > > Please give more details (kernel version, exact mtu...), because it
> > > should not happen !
> > > 
> > > # ifconfig lo mtu 100000
> > > # scp -6 vmlinux edumazet@ip6-localhost:/tmp
> > > Password: 
> > > vmlinux
> > > 100%   24MB  23.5MB/s   00:00    
> > > # ls -l /tmp/vmlinux
> > > -rwxr-xr-x 1 edumazet eng 24668915 Apr  9 14:37 /tmp/vmlinux
> > 
> > I couldn't test it on my development system with a recent net-next kernel
> > yet, but on my laptop with a distribution 3.13.9 kernel this happened too.
> 
> Oh well, it seems ip6_mtu() needs to cap mtu to max mtu for non
> jumbograms...
> 
> -	return mtu;
> +	return min_t(unsigned int, mtu, IP6_MAX_MTU);
> 
> #define IP6_MAX_MTU (65535 + 40)

I thought ip6_default_advmss will deal with this which correctly limits the
mtu. Capping dst_mtu would make it harder to implement jumbograms some day.

I'll look into it, thank you.

  reply	other threads:[~2014-04-10 13:28 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-09 19:42 Does IPv6 support Jumbograms ? Francois WELLENREITER
2014-04-09 20:35 ` Hannes Frederic Sowa
2014-04-09 21:41 ` Eric Dumazet
2014-04-09 22:35   ` Hannes Frederic Sowa
2014-04-10  0:36     ` Eric Dumazet
2014-04-10 13:28       ` Hannes Frederic Sowa [this message]
2014-04-10 14:01         ` Eric Dumazet
2014-04-10 23:54       ` Hannes Frederic Sowa
2014-04-11  0:40         ` Eric Dumazet
2014-04-11  1:42           ` [PATCH] ipv6: Limit mtu to 65572 bytes Eric Dumazet
2014-04-11  1:58             ` Eric Dumazet
2014-04-11  2:30             ` YOSHIFUJI Hideaki
2014-04-11  2:57               ` Hannes Frederic Sowa
2014-04-11  3:14                 ` Eric Dumazet
2014-04-11  8:40                   ` David Laight
2014-04-11  8:34                 ` David Laight
2014-04-11  3:20               ` Eric Dumazet
2014-04-11  3:26                 ` YOSHIFUJI Hideaki
2014-04-11  3:30                   ` YOSHIFUJI Hideaki
2014-04-11  3:30                 ` David Miller
2014-04-11  4:20                   ` Eric Dumazet
2014-04-11  4:23                 ` [PATCH v3] ipv6: Limit mtu to 65575 bytes Eric Dumazet
2014-04-11 13:22                   ` Hannes Frederic Sowa
2014-04-11 15:26                     ` Eric Dumazet
2014-04-11 20:48                   ` David Miller
2014-04-11  2:44             ` [PATCH] ipv6: Limit mtu to 65572 bytes Hannes Frederic Sowa

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140410132842.GI27255@order.stressinduktion.org \
    --to=hannes@stressinduktion.org \
    --cc=eric.dumazet@gmail.com \
    --cc=f.wellenreiter@gmail.com \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).