Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 14 Apr 2000 04:18:25 +1000 (EST)
From:      Bruce Evans <bde@zeta.org.au>
To:        Warner Losh <imp@village.org>
Cc:        Bill Fumerola <billf@chc-chimes.com>, Ustimenko Semen <semenu@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/pci if_tx.c if_txvar.h 
Message-ID:  <Pine.BSF.4.21.0004140403430.6368-100000@alphplex.bde.org>
In-Reply-To: <200004122036.OAA39907@harmony.village.org>

next in thread | previous in thread | raw e-mail | index | archive | help
> In message <20000412132155.J23367@jade.chc-chimes.com> Bill Fumerola writes:
> : On Tue, Apr 11, 2000 at 11:51:45PM -0700, Ustimenko Semen wrote:
> : > semenu      2000/04/11 23:51:45 PDT
> : > 
> : >   Modified files:
> : >     sys/pci              if_tx.c if_txvar.h 
> : >   Log:
> : >   Driver is now using miibus, and newbus.
> : >   
> : >   PR:		kern/17601
> : 
> : This breaks world (sys/modules/tx).

> It appears to need a shim define (it should likely be converted to
> newbus completely) as well as bpf.h.  I think the latter can be
> removed completely.

It still has "#if NBPF > 0", so removing the include of "bpf.h" would
break bpf support in all cases, not just for modules.  Most drivers
have been converted to support bpf unconditionally.  All the modularized
ones certainly had been converted.  Several drivers use the even more
bogus "#if NBPFILTER > 0", but its not clear that this breaks anything
because it seems to be mostly inside ifdef tangles.

Bruce



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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0004140403430.6368-100000>