lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 29 Sep 2013 17:45:46 +0200 From: Hannes Frederic Sowa <hannes@...essinduktion.org> To: Oussama Ghorbel <oghorbell@...il.com> Cc: "David S. Miller" <davem@...emloft.net>, Alexey Kuznetsov <kuznet@....inr.ac.ru>, James Morris <jmorris@...ei.org>, Hideaki YOSHIFUJI <yoshfuji@...ux-ipv6.org>, Patrick McHardy <kaber@...sh.net>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, ou.ghorbel@...il.com Subject: Re: [PATCH] IPv6: Allow the MTU of ipip6 tunnel to be set below 1280 On Sun, Sep 29, 2013 at 10:40:11AM +0100, Oussama Ghorbel wrote: > On Fri, Sep 27, 2013 at 6:03 PM, Hannes Frederic Sowa > <hannes@...essinduktion.org> wrote: > > Ok, let's go with one function per protocol type. Seems easier. > > > > It seems to get more hairy, because it depends on the tunnel driver if the > > prepended ip header is accounted in hard_header_len. :/ > > > > I don't know if it works out cleanly. Otherwise I would be ok if the checks > > just get repeated in ip6_tunnel and leave the rest as-is. > > > Yes, It will be the clean way to do it. Fine. :) > > > > Linux currently cannot create "jumbograms" (only the receiving side > > is supported). > > > I understand, but what are the benefit from this limit or the harm > from not specifying it? > Please check this comment from eth.c > > /** > * eth_change_mtu - set new MTU size > * @dev: network device > * @new_mtu: new Maximum Transfer Unit > * > * Allow changing MTU size. Needs to be overridden for devices > * supporting jumbo frames. > */ > int eth_change_mtu(struct net_device *dev, int new_mtu) Hmm, I cannot judge without the full patch. Will it be applicable to all net_devices or just ethernet ones? The name could be a bit misleading. Remindes me a lot of dev_set_mtu based on the signature, btw. > So wouldn't be a good idea to let our function open for jumbo frames...? Hm, we can document the fact that the function would needed to be updated in that case. But we should not allow to set a mtu which would require jumbograms currently. Greetings, Hannes -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists