Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 28 Mar 2001 14:12:21 +0300
From:      Peter Pentchev <roam@orbitel.bg>
To:        Luigi Rizzo <luigi@info.iet.unipi.it>
Cc:        cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG
Subject:   Re: cvs commit: src/sys/net if_ethersubr.c
Message-ID:  <20010328141221.D5524@ringworld.oblivion.bg>
In-Reply-To: <200103141314.OAA17745@info.iet.unipi.it>; from luigi@info.iet.unipi.it on Wed, Mar 14, 2001 at 02:14:56PM %2B0100
References:  <20010314115412.C387@ringworld.oblivion.bg> <200103141314.OAA17745@info.iet.unipi.it>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Mar 14, 2001 at 02:14:56PM +0100, Luigi Rizzo wrote:
> > On Tue, Mar 13, 2001 at 02:00:33PM -0800, Luigi Rizzo wrote:
> > > luigi       2001/03/13 14:00:33 PST
> > > 
> > >   Modified files:        (Branch: RELENG_4)
> > >     sys/net              if_ethersubr.c 
> > >   Log:
> > >   Fix breakage in bridging introduced in 1.70.2.13:
> > >   when doing bridging, we want to receive packets from all
> > >   interfaces in the same cluster, so the MAC address check
> > >   does not apply (it is already done in the bridging functions).
> > >   
> > >   May I ask, for the future, a bit more thinking to the side effects
> > >   of your changes before committing them ???
> > >   
> > >   Approved-by: jkh
> > 
> > Should this fix PR kern/25630?  I'm asking because the submitter could
> > only test it on a production machine, where buildworlds/reinstalls are
> > somewhat expensive..
> 
> it might. it is a one-line change so you only need to reinstall
> the kernel to try.
> Please let me know if it fixes the problem.

I just closed the PR after the originator told me the problem was fixed.
Thanks!

G'luck,
Peter

-- 
When you are not looking at it, this sentence is in Spanish.

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010328141221.D5524>