From owner-cvs-all Wed Mar 28 3:13:51 2001 Delivered-To: cvs-all@freebsd.org Received: from ringworld.nanolink.com (ringworld.nanolink.com [195.24.48.13]) by hub.freebsd.org (Postfix) with SMTP id 4023137B71E for ; Wed, 28 Mar 2001 03:13:32 -0800 (PST) (envelope-from roam@orbitel.bg) Received: (qmail 7922 invoked by uid 1000); 28 Mar 2001 11:12:21 -0000 Date: Wed, 28 Mar 2001 14:12:21 +0300 From: Peter Pentchev To: Luigi Rizzo 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> Mail-Followup-To: Luigi Rizzo , cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG References: <20010314115412.C387@ringworld.oblivion.bg> <200103141314.OAA17745@info.iet.unipi.it> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <200103141314.OAA17745@info.iet.unipi.it>; from luigi@info.iet.unipi.it on Wed, Mar 14, 2001 at 02:14:56PM +0100 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG 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