Date: Wed, 19 Aug 2009 11:40:14 -0700 From: Jack Vogel <jfvogel@gmail.com> To: Xin LI <delphij@freebsd.org> Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-8@freebsd.org Subject: Re: svn commit: r196387 - in stable/8/sys: . dev/e1000 Message-ID: <2a41acea0908191140n7ac726b6x46ae428e3490e0de@mail.gmail.com> In-Reply-To: <200908191808.n7JI8pNM061610@svn.freebsd.org> References: <200908191808.n7JI8pNM061610@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Thanks for doing this for me Xin. Jack On Wed, Aug 19, 2009 at 11:08 AM, Xin LI <delphij@freebsd.org> wrote: > Author: delphij > Date: Wed Aug 19 18:08:50 2009 > New Revision: 196387 > URL: http://svn.freebsd.org/changeset/base/196387 > > Log: > MFC r196386: > > Temporarily enhance em(4) and igb(4) hack to take account for IFF_NOARP. > Without this changeset there will be no way to prevent these NICs from > sending ARP, which is harmful in server farms that is configured as > "Direct Server Return" behind a load balancer. > > A better fix would remove the whole hack completely but it would be > later than 8.0-RELEASE. > > Reviewed by: jfv, yongari > Approved by: re (kib) > > Modified: > stable/8/sys/ (props changed) > stable/8/sys/dev/e1000/if_em.c > stable/8/sys/dev/e1000/if_igb.c > > Modified: stable/8/sys/dev/e1000/if_em.c > > ============================================================================== > --- stable/8/sys/dev/e1000/if_em.c Wed Aug 19 17:59:41 2009 > (r196386) > +++ stable/8/sys/dev/e1000/if_em.c Wed Aug 19 18:08:50 2009 > (r196387) > @@ -1204,7 +1204,8 @@ em_ioctl(struct ifnet *ifp, u_long comma > em_init_locked(adapter); > EM_CORE_UNLOCK(adapter); > } > - arp_ifinit(ifp, ifa); > + if (!(ifp->if_flags & IFF_NOARP)) > + arp_ifinit(ifp, ifa); > } else > #endif > error = ether_ioctl(ifp, command, data); > > Modified: stable/8/sys/dev/e1000/if_igb.c > > ============================================================================== > --- stable/8/sys/dev/e1000/if_igb.c Wed Aug 19 17:59:41 2009 > (r196386) > +++ stable/8/sys/dev/e1000/if_igb.c Wed Aug 19 18:08:50 2009 > (r196387) > @@ -952,7 +952,8 @@ igb_ioctl(struct ifnet *ifp, u_long comm > igb_init_locked(adapter); > IGB_CORE_UNLOCK(adapter); > } > - arp_ifinit(ifp, ifa); > + if (!(ifp->if_flags & IFF_NOARP)) > + arp_ifinit(ifp, ifa); > } else > #endif > error = ether_ioctl(ifp, command, data); >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?2a41acea0908191140n7ac726b6x46ae428e3490e0de>