cvs commit: src/share/man/man9 ifnet.9 src/sys/compat/ndis kern_ndis.c subr_ndis.c src/sys/contrib/altq/altq altq_rio.c src/sys/contrib/dev/oltr if_oltr.c if_oltr_pci.c if_oltrvar.h src/sys/contrib/pf/net if_pflog.c if_pflog.h if_pfsync.c ...

[Brooks Davis]

On Fri, Jun 10, 2005 at 02:28:57PM -0700, Julian Elischer wrote:
> 
> 
> Brooks Davis wrote:
> 
> >On Fri, Jun 10, 2005 at 05:08:18PM -0400, Andrew Gallatin wrote:
> > 
> >
> >>
> >>What is the appropriate way to detach an interface now?  I used to
> >>call just ether_ifdetach(ifp); Now I call both if_detach and if_free.
> >>Eg:
> >>
> >>   ether_ifdetach(ifp);
> >>   if_free(ifp);
> >>   
> >>
> >
> >This is correct.
> 
> why does if_ether_detatch not call if_free()?

Symmetry, but importantly, because some drivers change their if_type and
this messing up the basic call to if_free.  It's possible that in the
future we'll want to move this into if_detach.

-- Brooks

-- 
Any statement of the form "X is the one, true Y" is FALSE.
PGP fingerprint 655D 519C 26A7 82E7 2529  9BF0 5D8E 8BE9 F238 1AD4
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-all/attachments/20050610/047fc34c/attachment.bin