Date: Tue, 11 Apr 2017 09:51:35 -0700 (PDT) From: "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net> To: "Andrey V. Elsukov" <ae@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r316699 - head/sys/net Message-ID: <201704111651.v3BGpZi9075424@pdx.rh.CN85.dnsmgr.net> In-Reply-To: <201704110856.v3B8uJq5039806@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> Author: ae > Date: Tue Apr 11 08:56:18 2017 > New Revision: 316699 > URL: https://svnweb.freebsd.org/changeset/base/316699 > > Log: > Do not adjust interface MTU automatically. Leave this task to the system > administrator. > > This restores the behavior that was prior to r274246. > > No objection from: #network > MFC after: 2 weeks > Differential Revision: https://reviews.freebsd.org/D10215 > > Modified: > head/sys/net/if_gre.c > head/sys/net/if_gre.h > head/sys/net/if_me.c > > Modified: head/sys/net/if_gre.c > ============================================================================== > --- head/sys/net/if_gre.c Tue Apr 11 08:29:12 2017 (r316698) > +++ head/sys/net/if_gre.c Tue Apr 11 08:56:18 2017 (r316699) > @@ -88,7 +88,7 @@ __FBSDID("$FreeBSD$"); > #include <machine/in_cksum.h> > #include <security/mac/mac_framework.h> > > -#define GREMTU 1500 > +#define GREMTU 1476 I would of thought that this was #define GREMTU ETHERMTU and now should probably be: #define GREMTU ETHERMTU - gre_hlen; Or what ever the approprite sizeof(foo) is; Isn't this arguably wrong in the face of JumboFrames? > static const char grename[] = "gre"; > static MALLOC_DEFINE(M_GRE, grename, "Generic Routing Encapsulation"); > static VNET_DEFINE(struct mtx, gre_mtx); > @@ -173,7 +173,7 @@ gre_clone_create(struct if_clone *ifc, i > GRE2IFP(sc)->if_softc = sc; > if_initname(GRE2IFP(sc), grename, unit); > > - GRE2IFP(sc)->if_mtu = sc->gre_mtu = GREMTU; > + GRE2IFP(sc)->if_mtu = GREMTU; > GRE2IFP(sc)->if_flags = IFF_POINTOPOINT|IFF_MULTICAST; > GRE2IFP(sc)->if_output = gre_output; > GRE2IFP(sc)->if_ioctl = gre_ioctl; > @@ -231,7 +231,8 @@ gre_ioctl(struct ifnet *ifp, u_long cmd, > /* XXX: */ > if (ifr->ifr_mtu < 576) This is another magic constant that should be in a #define. > return (EINVAL); > - break; > + ifp->if_mtu = ifr->ifr_mtu; > + return (0); > case SIOCSIFADDR: > ifp->if_flags |= IFF_UP; > case SIOCSIFFLAGS: > @@ -255,12 +256,6 @@ gre_ioctl(struct ifnet *ifp, u_long cmd, > } > error = 0; > switch (cmd) { > - case SIOCSIFMTU: > - GRE_WLOCK(sc); > - sc->gre_mtu = ifr->ifr_mtu; > - gre_updatehdr(sc); > - GRE_WUNLOCK(sc); > - goto end; > case SIOCSIFPHYADDR: > #ifdef INET6 > case SIOCSIFPHYADDR_IN6: > @@ -549,7 +544,6 @@ gre_updatehdr(struct gre_softc *sc) > } else > sc->gre_oseq = 0; > gh->gre_flags = htons(flags); > - GRE2IFP(sc)->if_mtu = sc->gre_mtu - sc->gre_hlen; > } > > static void > > Modified: head/sys/net/if_gre.h > ============================================================================== > --- head/sys/net/if_gre.h Tue Apr 11 08:29:12 2017 (r316698) > +++ head/sys/net/if_gre.h Tue Apr 11 08:56:18 2017 (r316699) > @@ -69,7 +69,6 @@ struct gre_softc { > uint32_t gre_oseq; > uint32_t gre_key; > uint32_t gre_options; > - uint32_t gre_mtu; > u_int gre_fibnum; > u_int gre_hlen; /* header size */ > union { > > Modified: head/sys/net/if_me.c > ============================================================================== > --- head/sys/net/if_me.c Tue Apr 11 08:29:12 2017 (r316698) > +++ head/sys/net/if_me.c Tue Apr 11 08:56:18 2017 (r316699) > @@ -66,7 +66,7 @@ __FBSDID("$FreeBSD$"); > #include <machine/in_cksum.h> > #include <security/mac/mac_framework.h> > > -#define MEMTU 1500 > +#define MEMTU (1500 - sizeof(struct mobhdr)) 1500 -> ETHERMTU? > static const char mename[] = "me"; > static MALLOC_DEFINE(M_IFME, mename, "Minimal Encapsulation for IP"); > static VNET_DEFINE(struct mtx, me_mtx); > @@ -186,7 +186,7 @@ me_clone_create(struct if_clone *ifc, in > ME2IFP(sc)->if_softc = sc; > if_initname(ME2IFP(sc), mename, unit); > > - ME2IFP(sc)->if_mtu = MEMTU - sizeof(struct mobhdr); > + ME2IFP(sc)->if_mtu = MEMTU;; > ME2IFP(sc)->if_flags = IFF_POINTOPOINT|IFF_MULTICAST; > ME2IFP(sc)->if_output = me_output; > ME2IFP(sc)->if_ioctl = me_ioctl; > @@ -236,7 +236,7 @@ me_ioctl(struct ifnet *ifp, u_long cmd, > case SIOCSIFMTU: > if (ifr->ifr_mtu < 576) > return (EINVAL); > - ifp->if_mtu = ifr->ifr_mtu - sizeof(struct mobhdr); > + ifp->if_mtu = ifr->ifr_mtu; > return (0); > case SIOCSIFADDR: > ifp->if_flags |= IFF_UP; > > -- Rod Grimes rgrimes@freebsd.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201704111651.v3BGpZi9075424>