Date: 26 Feb 2003 12:16:06 -0500 From: Joe Marcus Clarke <marcus@marcuscom.com> To: Julian Elischer <julian@elischer.org> Cc: Bill Paul <wpaul@FreeBSD.ORG>, src-committers@FreeBSD.ORG, cvs-src@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/netgraph ng_fec.c Message-ID: <1046279766.311.4.camel@gyros> In-Reply-To: <Pine.BSF.4.21.0302252331450.78515-100000@InterJet.elischer.org> References: <Pine.BSF.4.21.0302252331450.78515-100000@InterJet.elischer.org>
index | next in thread | previous in thread | raw e-mail
[-- Attachment #1 --] On Wed, 2003-02-26 at 02:41, Julian Elischer wrote: > On Tue, 25 Feb 2003, Bill Paul wrote: > > > wpaul 2003/02/25 22:38:54 PST > > > > Modified files: > > sys/netgraph ng_fec.c > > Log: > > Attempt to make the ng_fec module play nice with BPF again. Things have > > changed since this code was written: > > > > - The ng_ether_input_p hook only accepts two arguments now: the pointer > > to the ether header structure is gone. > > The problem is that you should not be using ng_ether_input_p at all > that is a private interface between ng_ether.c and if_ethersubr.c. > You should be connecting a hook to ng_ether.c which will in turn > use ng_ether_input_p. > I have on my list (right after my real job(TM) and kernel threading > and some netgraph cleanups) to alter ng_fec to use the official > interface. > i.e. yu should be able to say: > ngctl fxp0: mkpeer fec lower lower1 > ngctl name fxp0:lower fec1 > ngctl fxp1: connect fec1: lower lower2 > ngctl fxp0: connect fec1: upper upper > > and hook an fec module onto two ethernet interfaces. > (using fxp0 as the upper end) > > as it is, having ng_fec loaded in the kernel produces an instant > coredump if ng_ether is also loaded as they fight over ng_ether's > private hook into the ethernet code. > > As I said.. It's on my list .. but don't hold your breath. Well, whenever you need someone to test, I still have my -CURRENT/-STABLE fec test machine setup. I haven't tested Bill's latest patch, but last I did check, -CURRENT was still panicking under any load iff ng_fec was loaded as a module. Joe > > > > > > > - It's no longer necessary to cons up a fake ether header before passing > > incoming packets to BPF_MTAP(). > > > > ng_fec_input() has been modified to account for these two changes. > > Running tcpdump on fec0 should work now. > > > > PR: kern/46720 > > > > Revision Changes Path > > 1.4 +6 -16 src/sys/netgraph/ng_fec.c > > > > > To Unsubscribe: send mail to majordomo@FreeBSD.org > with "unsubscribe cvs-src" in the body of the message -- PGP Key : http://www.marcuscom.com/pgp.asc [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (FreeBSD) iD8DBQA+XPZWb2iPiv4Uz4cRArBzAJ4/XHcQoLys/4NndDyCLPnfauNVswCeJl0k hFlJMGjl9uUJ+xgOOWHVZAM= =vPuu -----END PGP SIGNATURE-----help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1046279766.311.4.camel>