Re: if_bridge: unused 'disable' parameter in bridge_stop()
- In reply to: Roy Marples : "Re: if_bridge: unused 'disable' parameter in bridge_stop()"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sun, 21 Jun 2026 15:14:01 UTC
> On Jun 20, 2026, at 4:18 PM, Roy Marples <roy@marples.name> wrote: > > From: Zhenlei Huang <zlei.huang@gmail.com> > To: "Aaron Espinoza"<acesp25@freebsd.org> > Cc: <freebsd-net@freebsd.org> > Date: Sat, 20 Jun 2026 07:41:33 +0100 > Subject: Re: if_bridge: unused 'disable' parameter in bridge_stop() >> From git history, the parameter was introduced while importing from NetBSD. >> >> The purpose of that parameter is to match the requirement of NetBSD's ifnet protocol `ifp->if_stop` , >> ``` >> ifp->if_stop = bridge_stop; >> ``` >> >> That parameter is not even used in NetBSD, see https://github.com/NetBSD/src/blob/trunk/sys/net/if_bridge.c . > > NetBSD most definitely still uses if_stop > https://github.com/NetBSD/src/blob/ff256e296a0ab1d32dacadd29b55a40b933b3014/sys/net/if.c#L2827 Oops, I might mislead. Let me try to make it clear. Yes, if_stop is still used in NetBSD. What I said > That parameter is not event used meant `That parameter of bridge_stop() is not event used by bridge(4) in NetBSD`. > > Roy > Best regards, Zhenlei