Skip site navigation (1)Skip section navigation (2)
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>