BPF extensions

Joerg Sonnenberger joerg at britannica.bec.de
Tue Jan 25 11:04:14 PST 2005


On Mon, Jan 24, 2005 at 12:19:19PM -0800, Jeffrey Hsu wrote:
> +void
> +bpfattach2(struct ifnet *ifp, u_int dlt, u_int hdrlen, struct bpf_if 
> **driverp)
> 
> Same here.  Use a more descriptive name than attach2.

I've used bpfattach_dlt, that should perfectly reflect what the
function is about.

The patch (http://leaf.dragonflybsd.org/~joerg/bpf.diff) is updated
to reflect this change, the new comments for bpf_[m]_tap* and the
macros. The /* XXX */ comment has been replaced by the spl protection,
it's needed.

More comments?

Joerg





More information about the Submit mailing list