From owner-freebsd-net  Tue Jul 11 16:56:32 2000
Delivered-To: freebsd-net@freebsd.org
Received: from mail.interware.hu (mail.interware.hu [195.70.32.130])
	by hub.freebsd.org (Postfix) with ESMTP
	id 4DD7337B97D; Tue, 11 Jul 2000 16:56:23 -0700 (PDT)
	(envelope-from julian@elischer.org)
Received: from harare-34.budapest.interware.hu ([195.70.50.34] helo=jules.elischer.org)
	by mail.interware.hu with smtp (Exim 3.12 #1 (Debian))
	id 13C9ss-0006ou-00; Wed, 12 Jul 2000 01:56:14 +0200
Message-ID: <396BB40C.41C67EA6@elischer.org>
Date: Tue, 11 Jul 2000 16:55:56 -0700
From: Julian Elischer <julian@elischer.org>
X-Mailer: Mozilla 3.04Gold (X11; I; FreeBSD 5.0-CURRENT i386)
MIME-Version: 1.0
To: Archie Cobbs <archie@whistle.com>
Cc: freebsd-net@freebsd.org, csg@waterspout.com, ache@freebsd.org,
	wpaul@freebsd.org, imp@village.org
Subject: Re: ether_ifattach()/ether_ifdetach() patch
References: <200007112346.QAA41027@bubba.whistle.com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Sender: owner-freebsd-net@FreeBSD.ORG
Precedence: bulk
X-Loop: FreeBSD.org

Archie Cobbs wrote:
> 
> Julian Elischer writes:
> > > To any interested parties, here is a patch to review.
> > >
> > > This changes all ethernet drivers to only call ether_ifattach()
> > > and ether_ifdetach(), consolidating some common actions, namely,
> > > calls to if_attach(), bpfattach(), ng_ether_attach(), if_detach(),
> > > bpfdetach(), and ng_ether_detach().
> > >
> > > This also fixes the problem where detaching an ethernet interface
> > > previously used for ng_ether(4) would panic (which resulted from
> > > fixing the build to not require 'device ether' (which resulted
> > > from making ng_ether(4) a loadable KLD)).
> > >
> > >   ftp://ftp.whistle.com/pub/archie/misc/ETHERATTACH.patch.1
> >
> > The constant 1 (2nd argument to ether_attach) should be
> > defined as something meaningful in the ethernet include file.
> > e.g. USE_BPF vs NO_BPF
> 
> Thanks for the comment.. here's a revised version:
> 
>   ftp://ftp.whistle.com/pub/archie/misc/ETHERATTACH.patch.2
ok so you use ETHER_BPF_SUPPORTED...

but I never saw a case where it was not supported..
why is it needed?

> 
> -Archie
> 
> ___________________________________________________________________________
> Archie Cobbs   *   Whistle Communications, Inc.  *   http://www.whistle.com

-- 
      __--_|\  Julian Elischer
     /       \ julian@elischer.org
    (   OZ    ) World tour 2000
     ;_.---._/  presently in:  Budapest
            v


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